[PATCH] D59048: Add AIX Target Info

Andus Yu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 8 11:18:59 PST 2019


andusy marked 7 inline comments as done.
andusy added inline comments.


================
Comment at: clang/lib/Basic/Targets/OSTargets.h:641
+    }
+  }
+
----------------
hubert.reinterpretcast wrote:
> D18360 sets `_THREAD_SAFE` to `1` when `-pthread` is specified. I believe that to be correct. I believe whether or not `-pthread` is taken to be the default on the platform is a separate consideration.
I've added a test in `clang/test/Preprocessor/init.c` to check if the macro is defined when `-pthread` is enabled. 


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59048/new/

https://reviews.llvm.org/D59048





More information about the cfe-commits mailing list