[PATCH] D100424: [RISCV] Share RVInstIShift and RVInstIShiftW instruction format classes with the B extension.
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 15 11:10:03 PDT 2021
craig.topper added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoB.td:130
- opcodestr, "$rd, $rs1, $shamt"> {
- bits<6> shamt;
-
----------------
frasercrmck wrote:
> So this was a 6-bit field even though the instruction only takes 4/5 bits through `shfl_uimm`?
Yeah. Maybe it was trying to cover for RV128 because that's how the spec is written. But the other shifts just put a 0 where the extra bit of shift amount goes so if it was it wasn't consistent.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100424/new/
https://reviews.llvm.org/D100424
More information about the llvm-commits
mailing list