[PATCH] D108694: [RISCV] Add the zvl extension according to the v1.0-rc1 spec

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added a comment.

I also don't think we're ready/able to support `Zvl32b` for the reasons stated above. I think the smoothest path forward would be, as @HsiangKai suggests, to only support VLEN>=64 and revisit 32. Supporting 32 is (I think) a much larger thing owing to our codegen assumptions on `VLEN` and `ELEN`. And, as @craig.topper said, there are a few little things that would probably be best prepared before this patch, replacing some of the calls to `hasStdExtV` at least (`hasVVectors`, for codegen purposes?). It sounds like we're unsure on some of the finer details about how the extensions interact.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108694