pengfei added inline comments.
================
Comment at: clang/include/clang/CodeGen/CGFunctionInfo.h:744
+ void setMaxVectorWidth(unsigned Width) {
+ MaxVectorWidth = llvm::Log2_32(Width) + 1;
+ }
----------------
craig.topper wrote:
> Are you assuming Width is a power of 2? Should we assert that?
Good question! I assumed it, but I found it's not true for Clang, although ICC
and GCC error for it.
Maybe we should diagnose it too? Anyway, I added an assert for it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122104/new/
https://reviews.llvm.org/D122104
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
