| From: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
|---|---|
| To: | Zsolt Parragi <zsolt(dot)parragi(at)percona(dot)com> |
| Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org, Andres Freund <andres(at)anarazel(dot)de>, Peter Eisentraut <peter(at)eisentraut(dot)org> |
| Subject: | Re: meson vs. llvm bitcode files |
| Date: | 2026-07-06 13:37:40 |
| Message-ID: | CAN55FZ2V+X=Wgyr2U0C-=hH1RWELmuQcqV523o0R1Yi22K4FCg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
Sorry, I didn't notice your reply as it is sent to -hackers list only.
Thank you for looking into this!
On Fri, 3 Jul 2026 at 00:50, Zsolt Parragi <zsolt(dot)parragi(at)percona(dot)com> wrote:
>
> Everything seems to be properly working with the new version in my
> tests, I only have a few minor comments:
>
> + <para>
> + When <productname>PostgreSQL</productname> is built with
> + <productname>Meson</productname>, it installs a
> + <literal>pkg-config</literal> file named
> + <literal>postgresql-&majorversion;-llvm-jit-bitcode.pc</literal>. It is
> + used internally to emit <acronym>LLVM</acronym> <acronym>JIT</acronym>
> + bitcode for the server and for selected
> + <filename>contrib</filename> modules, and it collects the same
> + compilation flags that are used to build the server itself.
> + </para>
>
> Do we want to install these pc files even with -Dllvm=disabled?
I think we want to install them as these pc files can be used for
building extensions.
> diff --git a/src/tools/irlink b/src/tools/irlink
> new file mode 100644
> index 00000000000..6a03e4f5695
> --- /dev/null
> +++ b/src/tools/irlink
>
> Shouldn't this file committed as executable?
Yes, done.
> +import shutil
> +import subprocess
> +import sys
>
> Is shutil and sys used in the script?
They are not used, removed.
v14 is attached.
--
Regards,
Nazir Bilal Yavuz
Microsoft
| Attachment | Content-Type | Size |
|---|---|---|
| v14-0001-meson-Add-postgresql-version-llvm-jit-bitcode.pc.patch | text/x-patch | 5.3 KB |
| v14-0002-meson-Add-docs-for-postgresql-version-llvm-jit-b.patch | text/x-patch | 5.0 KB |
| v14-0003-meson-Add-architecture-for-LLVM-bitcode-emission.patch | text/x-patch | 10.0 KB |
| v14-0004-meson-Add-LLVM-bitcode-emissions-for-contrib-lib.patch | text/x-patch | 3.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Xuneng Zhou | 2026-07-06 13:49:26 | Re: Implement waiting for wal lsn replay: reloaded |
| Previous Message | Tomas Vondra | 2026-07-06 13:34:57 | Re: hashjoins vs. Bloom filters (yet again) |