diff options
| author | Tom Lane | 2018-11-19 17:43:05 +0000 |
|---|---|---|
| committer | Tom Lane | 2018-11-19 17:43:20 +0000 |
| commit | 640a4ba052c2dd7aaf600ca13b6d3afca60adc05 (patch) | |
| tree | eeb37bec4f377a91cefd7c24a83e327c1e7054bb /configure.in | |
| parent | 6e5f8d489acccdc50a35a1b7db8e72b5ad579253 (diff) | |
Postpone LLVM-related uses of AC_CHECK_DECLS.
Calling AC_CHECK_DECLS before we've finished setting up the compiler's
CFLAGS seems like a pretty risky proposition, especially now that the
first use of that macro will result in a test to see whether the compiler
gives warning or error for undeclared built-in functions. That answer
could very easily get changed later than where PGAC_LLVM_SUPPORT is
called; furthermore, it's hardly unlikely that flags such as -D_GNU_SOURCE
could change visibility of declarations. Hence, be a little less cavalier
about where to do LLVM-related tests. This results in v11 and HEAD doing
the warning-or-error check at the same place in the script as older
branches are doing it, which seems like a good thing.
Per further thought about commits 0b59b0e8b and 16fbac39f.
Diffstat (limited to 'configure.in')
| -rw-r--r-- | configure.in | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/configure.in b/configure.in index 2b71019fd1..e5123ac122 100644 --- a/configure.in +++ b/configure.in @@ -1832,6 +1832,10 @@ if test "$with_icu" = yes; then CPPFLAGS=$ac_save_CPPFLAGS fi +if test "$with_llvm" = yes; then + PGAC_CHECK_LLVM_FUNCTIONS() +fi + # Lastly, restore full LIBS list and check for readline/libedit symbols LIBS="$LIBS_including_readline" |