kito-cheng accepted this revision.
kito-cheng added a comment.
This revision is now accepted and ready to land.
LGTM, and verified with internal testsuite :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126742/new/
https://reviews.llvm.org/D126742
StephenFan added inline comments.
Herald added a subscriber: nlopes.
Comment at: clang/lib/Sema/SemaRVVLookup.cpp:378-388
+} else {
+ if (IsPrototypeDefaultTU) {
+DefaultPolicy = Policy::TU;
+if (HasPolicy)
+ BuiltinName += "_tu";
+ } else {
fakepaper56 added inline comments.
Comment at: clang/lib/Sema/SemaRVVLookup.cpp:329
+
+ auto appendPolicySuffix = [&](std::string suffix) {
+Name += suffix;
Using `const std::string &suffix`.
Comment at: clang/lib/Support/RISCVVIntrinsicUt
