Revert "Use -isystem for LLVM include directories" This reverts commit 704ef84ef232e02e241071b2faf0541d155c6b62.
The configure solution in that commit doesn't work if llvm-config returns a system directory such as /usr/include. The resulting -isystem /usr/include puts the C headers before the C++ headers in the include file order, which is not supposed to be done and causes failures. (Using -I /usr/include doesn't have that problem because it is handled specially by the compiler.) Per buildfarm members hippopotamus and jay. The meson solution does work because it specifically filters out system includes. This logic could theoretically be ported to configure, but that would be a separate undertaking. Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/7a0aa99e51c7a3263e6ee45f19f270ddae430a92 Modified Files -------------- config/llvm.m4 | 7 ++----- configure | 7 ++----- meson.build | 5 +---- 3 files changed, 5 insertions(+), 14 deletions(-)
