[PATCH] D117854: [RISCV] Refactor Zve* extensions.

[Yueh-Ting Chen via Phabricator via llvm-commits]

eopXD added a comment.

> In my opinion, FeatureStdExtV just means that we have the V extension. In the instruction level, the V extension includes all the Zve* extensions. But they still have different using space which V is for application processors and Zve* is for embedded processors. Maybe it's better to not mix these two things?

Yes I agree with you that they are different and shouldn't be mixed.
On the other hand, if we want to guard `vmulh` out of `Zve*` extension shouldn't we use `FeatureStdExtV`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117854