[PATCH] D82035: [PowerPC] Add Sema checks for MMA types
Victor Huang via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 4 15:21:09 PST 2020
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
More information about the cfe-commits
mailing list