[PATCH] D120009: [RISCV][NFC] Add sched and adjust predicates for some instructions in Zb extension

WangLian via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 17 19:18:23 PST 2022


Jimerlife added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoZb.td:860
 
+let Predicates = [HasStdExtZbpOrZbkb, IsRV32] in {
 // We treat zip and unzip as separate instructions, so match it directly.
----------------
craig.topper wrote:
> Is this not a functional change for RV64 because ZIP_RV32 will put 0b01111 in bits 24:20 which matches the 15 that was requested in the pattern?
This change will not affect RV64. where defined ZIP_RV32 and UNZIP_RV32 also add "IsRV32" constrain.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120009



More information about the llvm-commits mailing list