[PATCH] D98180: [RISCV] Fix vector load/store whole register inst encoding

Fraser Cormack via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 8 07:43:55 PST 2021


frasercrmck created this revision.
frasercrmck added reviewers: HsiangKai, craig.topper, evandro, rogfer01.
Herald added subscribers: vkmr, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, simoncook, johnrusso, rbar, asb, hiraditya.
frasercrmck requested review of this revision.
Herald added subscribers: llvm-commits, MaskRay.
Herald added a project: LLVM.

The `nf` field of the RVV vector load/store whole register instructions
(Section 7.9) "encodes the number of vector registers to transfer,
numbered successively after the base". Or, the log2 of the number of
registers to transfer. We were previously encoding it naively as
1/2/4/8, the last of which doesn't even fit in the 3-bit field.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D98180

Files:
  llvm/lib/Target/RISCV/RISCVInstrInfoV.td
  llvm/test/MC/RISCV/rvv/aliases.s
  llvm/test/MC/RISCV/rvv/load.s
  llvm/test/MC/RISCV/rvv/store.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98180.329013.patch
Type: text/x-patch
Size: 9633 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210308/7c2a93b3/attachment.bin>


More information about the llvm-commits mailing list