From 5758685c9f1a51b0a77ae8f415a57670941f2c4b Mon Sep 17 00:00:00 2001 From: Andres Freund Date: Fri, 31 Aug 2018 16:56:11 -0700 Subject: [PATCH] Fix 8a934d677 for libc++ and make more include order resistant. The previous definition was used in C++ mode, which causes problems when using clang with libc++ (rather than libstdc++), due to bugs therein. So just avoid in C++ mode. A second problem is that depending on include order and implicit includes the previous definition did not guarantee that the current hack was effective by the time isinf was used, fix that by forcing math.h to be included. This can cause clang using builds, or gcc using ones with JIT enabled, to slow down noticably. It's likely that we at some point want a better solution for the performance problem, but while it's there it should better work. Reported-By: Steven Winfield Bug: #15270 Discussion: https://postgr.es/m/153116283147.1401.360416241833049560@wrigleys.postgresql.org Author: Andres Freund Backpatch: 11, like the previous commit. --- src/include/port.h | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/include/port.h b/src/include/port.h index 74a9dc4d4d..0ce72e50e5 100644 --- a/src/include/port.h +++ b/src/include/port.h @@ -347,15 +347,18 @@ extern int isinf(double x); /* * Glibc doesn't use the builtin for clang due to a *gcc* bug in a version * newer than the gcc compatibility clang claims to have. This would cause a - * *lot* of superfluous function calls, therefore revert when using clang. + * *lot* of superfluous function calls, therefore revert when using clang. In + * C++ there's issues with libc++ (not libstdc++), so disable as well. */ -#ifdef __clang__ +#if defined(__clang__) && !defined(__cplusplus) /* needs to be separate to not confuse other compilers */ #if __has_builtin(__builtin_isinf) +/* need to include before, to avoid getting overwritten */ +#include #undef isinf #define isinf __builtin_isinf #endif /* __has_builtin(isinf) */ -#endif /* __clang__ */ +#endif /* __clang__ && !__cplusplus*/ #endif /* !HAVE_ISINF */ #ifndef HAVE_MKDTEMP -- 2.40.0