]> 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:05 +0000 (12:43 -0500)
commit923f9c2def111e65a83f65d7caf75efb772314fc
tree60eea1b1667480d046eb529a48cabfdad8a0283d
parentdcd6200165ca583cc1b0d436774cf4c3cf7e642d
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