ZarkoCA added inline comments.
================ Comment at: clang/lib/CodeGen/TargetInfo.cpp:5244 llvm::Value *Address) const { - - return PPC64_initDwarfEHRegSizeTable(CGF, Address); + return PPC_initDwarfEHRegSizeTable(CGF, Address, /* Is64Bit*/ true, + /*IsAIX*/ false); ---------------- Missed to point out the extra space in `/* Is64Bit*/` previously. ================ Comment at: clang/lib/Driver/ToolChains/Clang.cpp:4541 options::OPT_msvr4_struct_return)) { - if (TC.getArch() != llvm::Triple::ppc) { + // TODO: We might want to consider enable these two options on AIX in the + // future. ---------------- s/enable/enabling/ CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79035/new/ https://reviews.llvm.org/D79035 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits