[PATCH] D112408: [WIP][RISCV] Add the zve extension according to the v1.0-rc2 spec
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 25 15:24:01 PDT 2021
craig.topper added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVSubtarget.h:141
+ // either v or zve* suppaort v instructions
+ bool hasStdExtV() const { return HasStdExtV || HasStdExtZve32x; }
+ bool hasStdExtZve32x() const { return HasStdExtZve32x; }
----------------
frasercrmck wrote:
> Is this correct? I thought we'd keep `hasStdExtV` as being the single-letter V extension, and Zve32x isn't that.
I just put up D112496 to stop using hasStdExtV everywhere.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112408/new/
https://reviews.llvm.org/D112408
More information about the llvm-commits
mailing list