NeHuang added inline comments.
================ Comment at: clang/lib/Sema/SemaChecking.cpp:10339 + if (Context.getTargetInfo().getTriple().isPPC64()) + CheckPPCMMAType(RetValExp->getType(), ReturnLoc); } ---------------- CheckPPCMMAType returns bool value. Do we need add it in the if condition (similar as `CheckNonNullExpr` above) and execute a Diag()? or I may miss something here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82035/new/ https://reviews.llvm.org/D82035 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
