[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