Re: Duplicated LLVMJitHandle->lljit assignment? - Mailing list pgsql-hackers

From Gurjeet Singh
Subject Re: Duplicated LLVMJitHandle->lljit assignment?
Date
Msg-id CABwTF4Vx=5JKVRSNZfthk8M+Quiq9DZQmzWsoo110+fxDj_BfA@mail.gmail.com
Whole thread Raw
In response to Duplicated LLVMJitHandle->lljit assignment?  (Matheus Alcantara <mths.dev@pm.me>)
Responses Re: Duplicated LLVMJitHandle->lljit assignment?
List pgsql-hackers
On Wed, Jul 12, 2023 at 5:22 PM Matheus Alcantara <mths.dev@pm.me> wrote:
>
> I was reading the jit implementation and I notice that the lljit field of
> LLVMJitHandle is being assigned twice on llvm_compile_module function, is this
> correct? I'm attaching a supposed fixes that removes  the second assignment. I
> ran meson test and all tests have pass.

-        handle->lljit = compile_orc;

LGTM.

Best regards,
Gurjeet
http://Gurje.et



pgsql-hackers by date:

Previous
From: "David G. Johnston"
Date:
Subject: Re: CommandStatus from insert returning when using a portal.
Next
From: Peter Smith
Date:
Subject: Re: [Patch] Use *other* indexes on the subscriber when REPLICA IDENTITY is FULL