[PATCH] D129013: [RISCV] Enable (scalable) vectorization by default

[Alex Bradbury via Phabricator via llvm-commits]

asb added a comment.

I think considering it user error if they're trying to run V binaries on a system that hasn't implemented the proper kernel-side support is probably the best option we have here.

The other option that came to mind was emitting a warning, but that warning could quickly become dated.

The fact that no existing upstream CPU definitions enable V support is a good point - this is probably something people will want to keep in mind when adding new definitions. It sounds like people will only get vectorisation if using a downstream toolchain or explicitly asking for V support, which minimises the risk of blowing up some previously working configuration.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129013/new/

https://reviews.llvm.org/D129013