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

Yueh-Ting Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 20 23:12:16 PST 2022


eopXD added a comment.

I see your point now.

In the v-spec describing for `Zve`:

> All Zve* extensions support all vector integer instructions (Section Vector Integer Arithmetic Instructions), except that the vmulh integer multiply variants that return the high word of the product (vmulh.vv, vmulh.vx, vmulhu.vv, vmulhu.vx, vmulhsu.vv, vmulhsu.vx) are not included for EEW=64 in Zve64*.

For V extension:

> The V extension supports all vector integer instructions (Section Vector Integer Arithmetic Instructions).

So we need to have something to express that V is enabled.

------

If I have parsed correctly in the above, I think we should have something that indicates V ext. is specified rather than adding `HasStdExtV` to the existing predicates.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117854



More information about the llvm-commits mailing list