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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits