]> granicus.if.org Git - postgresql/commit
Postpone LLVM-related uses of AC_CHECK_DECLS.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 19 Nov 2018 17:43:05 +0000 (12:43 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 19 Nov 2018 17:43:20 +0000 (12:43 -0500)
commit640a4ba052c2dd7aaf600ca13b6d3afca60adc05
treeeeb37bec4f377a91cefd7c24a83e327c1e7054bb
parent6e5f8d489acccdc50a35a1b7db8e72b5ad579253
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.
config/llvm.m4
configure
configure.in