Re: --with-llvm on 32-bit platforms? - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: --with-llvm on 32-bit platforms?
Date
Msg-id CA+hUKGLaNj7f47hTF5qLAmCYzz5yoR8PEz2OR_KNde2Jn6tLfg@mail.gmail.com
Whole thread Raw
In response to Re: --with-llvm on 32-bit platforms?  (Thomas Munro <thomas.munro@gmail.com>)
Responses Re: --with-llvm on 32-bit platforms?
List pgsql-hackers
On Tue, Sep 16, 2025 at 12:51 PM Thomas Munro <thomas.munro@gmail.com> wrote:
> On Tue, Sep 16, 2025 at 12:05 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> > Thomas Munro <thomas.munro@gmail.com> writes:
> > > Hmm.  We use TypeSizeT in generated IR for Datum, which is obviously
> > > incorrect in this configuration.
> >
> > Oh!  Yeah, that is surely broken now.

This patch seems to work OK here.  The deform code is a little tricky
as you have to think carefully about which places need TypeDatum and
which need TypeSizeT in llvmjit_deform.c, since the v_offp variable
really is size_t.  Tested on Debian 13 with i386 packages installed.

More changes would be needed if Datum is changed into a struct.

Attachment

pgsql-hackers by date:

Previous
From: torikoshia
Date:
Subject: Re: RFC: Logging plan of the running query
Next
From: Tom Lane
Date:
Subject: Re: --with-llvm on 32-bit platforms?