[PATCH] D108917: [PowerPC] Define __powerpc and __PPC macros

2021-09-01 Thread Jake Egan via Phabricator via cfe-commits
Jake-Egan updated this revision to Diff 369932. Jake-Egan added a comment. Define for AIX only. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108917/new/ https://reviews.llvm.org/D108917 Files: clang/lib/Basic/Targets/PPC.cpp clang/test/Prepr

[PATCH] D108917: [PowerPC] Define __powerpc and __PPC macros

2021-09-01 Thread Chris Bowler via Phabricator via cfe-commits
cebowleratibm added a comment. The XL compiler only defined this weird form of the macros on AIX, but not on the XL Linux on Power compiler. I think it's preferable that we define these macros only on AIX and in the source/commit messages indicate that we're only doing so for AIX XL C/C++ comp

[PATCH] D108917: [PowerPC] Define __powerpc and __PPC macros

2021-08-31 Thread Nemanja Ivanovic via Phabricator via cfe-commits
nemanjai added a comment. In D108917#2974490 , @cebowleratibm wrote: > I'd like to see the rationale for adding these forms of the macros in the > review and also in the extended commit message. These forms are being added > primarily because the AIX