[PATCH] D129013: [RISCV] Enable (scalable) vectorization by default
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 8 11:10:49 PDT 2022
craig.topper added a comment.
In D129013#3707331 <https://reviews.llvm.org/D129013#3707331>, @jrtc27 wrote:
> This is user error, plain and simple. If you say you have extension foo then the compiler should be free to use it. V isn't special; specifying +zba for example when your implementation lacks it has the exact same behaviour. If anything, leaving autovectorisation off would be more inconsistent.
What if we had a -mcpu that included V? Would specifying that CPU when the kernel doesn't support it also be user error?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129013/new/
https://reviews.llvm.org/D129013
More information about the llvm-commits
mailing list