[PATCH] D107244: [AIX] Define _ARCH_PPC64 macro for 32-bit

Hubert Tong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 4 08:33:16 PDT 2021


hubert.reinterpretcast added inline comments.


================
Comment at: clang/lib/Basic/Targets/PPC.cpp:260
+  } else {
+    if (getTriple().isOSAIX())
+      Builder.defineMacro("_ARCH_PPC64");
----------------
cebowleratibm wrote:
> A comment is warranted here to explain that __ARCH_PPC64 is only defined in 32-bit on AIX.
s/only/also/; ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107244



More information about the cfe-commits mailing list