[PATCH] D85324: [z/OS] Add z/OS Target and define macros

Anirudh Prasad via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 5 09:48:17 PDT 2020


anirudhp added inline comments.


================
Comment at: clang/lib/Basic/Targets/OSTargets.h:745
+    if (this->PointerWidth == 64) {
+      Builder.defineMacro("__64BIT__");
+    }
----------------
abhina.sreeskantharajan wrote:
> anirudhp wrote:
> > What about the `__LP64__` and the `_LP64` macros?
> These two are already added in the common code, so it would be redundant to add them again.
Alright Thanks! 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85324



More information about the cfe-commits mailing list