[PATCH] D107242: [AIX] Define __HOS_AIX__ macro
Chris Bowler via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 5 20:33:02 PDT 2021
cebowleratibm added inline comments.
================
Comment at: clang/lib/Basic/Targets/PPC.cpp:306
+ llvm::Triple HostTriple(llvm::sys::getProcessTriple());
+ if (HostTriple.isOSAIX() && getTriple().isOSAIX()) {
+ Builder.defineMacro("__HOS_AIX__");
----------------
suggest using a temp rather than unnecessarily declaring the local var:
if (Triple(sys::getProcessTriple()).isOSAIX() ...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107242/new/
https://reviews.llvm.org/D107242
More information about the cfe-commits
mailing list