[PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

Andrew Paprocki via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 18 12:00:00 PDT 2016


apaprocki added inline comments.

================
Comment at: lib/Basic/Targets.cpp:718
@@ +717,3 @@
+    Builder.defineMacro("_LONG_LONG");
+    Builder.defineMacro("_ALL_SOURCE");
+    Builder.defineMacro("_REENTRANT");
----------------
majnemer wrote:
> Are we really supposed to define this macro?  Does GCC define this?  I cannot find where it does so in the source.
I defined both `_ALL_SOURCE` and `_REENTRANT` because the `SolarisTargetInfo` defines them (`__EXTENSIONS__` is the Solaris equivalent of `_ALL_SOURCE`). If that is an oversight in the Solaris work, I'll remove `_ALL_SOURCE` here. GCC does not define it by default.


https://reviews.llvm.org/D18360





More information about the cfe-commits mailing list