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();
----------------
craig.topper wrote:
> 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.
I don't think this comment was addressed
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
