[PATCH] D94736: [RISCV] Change zext.w to be an alias of add.uw rd, rs1, x0 instead of packw.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 14 18:05:34 PST 2021


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

This didn't make it into the published 0.93 spec, but it was the
intention.

But it is in the tex source as of this commit
https://github.com/riscv/riscv-bitmanip/commit/d172f029c074d47026a0c0d0f12d8b475c86a472

This means zext.w now requires Zba. Not sure if we should still use
packw if Zbp is enabled and Zba isn't. I'll leave that for the future
when packw is closer to being final.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D94736

Files:
  llvm/lib/Target/RISCV/RISCV.td
  llvm/lib/Target/RISCV/RISCVInstrInfo.td
  llvm/lib/Target/RISCV/RISCVInstrInfoB.td
  llvm/test/CodeGen/RISCV/rv64Zba.ll
  llvm/test/CodeGen/RISCV/rv64Zbbp.ll
  llvm/test/MC/RISCV/rv64b-aliases-valid.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D94736.316819.patch
Type: text/x-patch
Size: 6706 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210115/46a00280/attachment.bin>


More information about the llvm-commits mailing list