On Wed, Jul 12, 2023 at 06:41:37PM -0700, Gurjeet Singh wrote:
> LGTM.
Indeed. It looks like a small thinko from 6c57f2e when support for
LLVM12 was added.
--
Michael
signature.asc
Description: PGP signature
On Wed, Jul 12, 2023 at 5:22 PM Matheus Alcantara 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
AlcantaraFrom 2a2c773b2437b2c491576db8d7ed6b6d1ba2c815 Mon Sep 17 00:00:00 2001
From: Matheus Alcantara
Date: Wed, 12 Jul 2023 18:57:52 -0300
Subject: [PATCH] Remove duplicated LLVMJitHandle->lljit assignment
---
src/backend/jit/llvm/llvmjit.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/