[PATCH] D156250: [RISCV] Handle seteq/setne conditions for CZERO_NEZ/CZERO_EQZ during isel.

Wang Pengcheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 20:27:58 PDT 2023


wangpc accepted this revision.
wangpc added a comment.
This revision is now accepted and ready to land.

LGTM in general.



================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoXVentana.td:32
 let Predicates = [IsRV64, HasVendorXVentanaCondOps] in {
 def : Pat<(XLenVT (riscv_czero_eqz GPR:$rs1, GPR:$rc)),
           (VT_MASKC GPR:$rs1, GPR:$rc)>;
----------------
These patterns are almost the same as `zicond`'s, is it possible to remove the redundancy?
(Just a weak suggestion, it's OK to ignore it)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156250



More information about the llvm-commits mailing list