Re: Consistently use palloc_object() and palloc_array() - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: Consistently use palloc_object() and palloc_array()
Date
Msg-id CA+hUKGLngd9cKHtTUuUdEo2eWEgUcZ_EQRbP55MigV2t_zTReg@mail.gmail.com
Whole thread Raw
In response to Re: Consistently use palloc_object() and palloc_array()  (Michael Paquier <michael@paquier.xyz>)
Responses Re: Consistently use palloc_object() and palloc_array()
List pgsql-hackers
On Wed, Dec 10, 2025 at 11:38 AM Michael Paquier <michael@paquier.xyz> wrote:
>             b_checkargnulls =
> -               palloc(sizeof(LLVMBasicBlockRef *) * op->d.func.nargs);
> +               palloc_array(LLVMBasicBlockRef *, op->d.func.nargs);
>
> This one in llvmjit_expr.c was causing a compilation failure.  I am
> not exactly sure why, but discarded for now.  I got a reproduction
> locally as well as in the CI.

I think the original code is wrong, it should have been
sizeof(LLVMBasicBlockRef)?  It'll be the same size anyway (these
LLVM*Ref types are just pointers), but that'd explain why the
transformation didn't compile.



pgsql-hackers by date:

Previous
From: Peter Geoghegan
Date:
Subject: Re: Trying out read streams in pgvector (an extension)
Next
From: Thomas Munro
Date:
Subject: Re: Solaris versus our NLS files