craig.topper added inline comments.
================
Comment at: clang/lib/Sema/SemaChecking.cpp:4969
+ const TargetInfo &TI = Context.getTargetInfo();
+ if ((Ty->isRVVBoolType() ||
+ Ty->isRVVType(/* Bitwidth */ 8, /* IsFloat */ false) ||
----------------
We don't to check exactly what type it is. We can do `if
(!TI.hasFeature("zve32x"))` after all the other checks. The caller already
checked it was an RVV type. We don't need to know anymore about the type, they
all require zve32x.
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
