[PATCH] D95146: [RISCV] Make v extension imply zvamo, zvlsseg

[Simon Cook via Phabricator via llvm-commits]

simoncook added a comment.

In D95146#2512871 <https://reviews.llvm.org/D95146#2512871>, @craig.topper wrote:

> Doesn't this mean that if you only enable zvlsseg, you'll be able to use the instruction in that extension but not the vsetvli instruction that you need to program the VL register?

@craig.topper To be honest, I'm not at all familiar with the v extension or any of the zv* extensions. I wrote this in response to D94931 <https://reviews.llvm.org/D94931> which says that V should imply zv* but not the other way around. Looking back now at D85069 <https://reviews.llvm.org/D85069> it looks there was some discussion suggesting which way around it should be, so it could be this patch is unnecessary and breaks stuff. Do you have any better understand of the vector spec as to which way round is correct?

@kito-cheng Similarly, do you know if the correct order of implication/requirements is well defined somewhere. If there are gcc/binutils patches what does it do here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95146