craig.topper added inline comments.
================
Comment at: clang/lib/Sema/SemaChecking.cpp:4967
+void Sema::checkRVVTypeSupport(QualType Ty, SourceLocation Loc, ValueDecl *D) {
+ const TargetInfo &TI = Context.getTargetInfo();
----------------
What about the bool types? Is it sufficient to check Zve32x is enabled after
all the other checks? The caller already checked for it being an RVV type so if
we are in the function the type is RVV.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153510/new/
https://reviews.llvm.org/D153510
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
