Re: some LLVM function checks missing in meson
Peter Eisentraut <peter@eisentraut.org>
From: Peter Eisentraut <peter@eisentraut.org>
To: Heikki Linnakangas <hlinnaka@iki.fi>,
pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2024-04-17T13:29:29Z
Lists: pgsql-hackers
On 13.04.24 10:25, Heikki Linnakangas wrote: >> Something like the below would appear to fix that: >> >> diff --git a/meson.build b/meson.build >> index 43fad5323c0..cdfd31377d1 100644 >> --- a/meson.build >> +++ b/meson.build >> @@ -2301,6 +2301,14 @@ decl_checks += [ >> ['pwritev', 'sys/uio.h'], >> ] >> >> +# Check presence of some optional LLVM functions. >> +if llvm.found() >> + decl_checks += [ >> + ['LLVMCreateGDBRegistrationListener', 'llvm-c/ExecutionEngine.h'], >> + ['LLVMCreatePerfJITEventListener', 'llvm-c/ExecutionEngine.h'], >> + ] >> +endif >> + >> foreach c : decl_checks >> func = c.get(0) >> header = c.get(1) >> >> I don't know what these functions do, but the symbols are used in the >> source code. Thoughts? > > +1. I also don't know what they do, but clearly the configure and meson > checks should be in sync. Committed that, too.
Commits
-
meson: Add some missing LLVM function checks
- 5165d43680da 17.0 landed
-
Remove dead code
- ca89db5f9d0b 17.0 landed
-
LLVMJIT: Adapt to API changes in gdb and perf support.
- e9a9843e1380 12.0 cited