[PATCH] D158976: [RISCV] Add isCommutable for pseudos without merge operand

Wang Pengcheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 28 20:20:28 PDT 2023


wangpc marked an inline comment as done.
wangpc added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td:2264
     defvar mx = m.MX;
-    let VLMul = m.value in {
+    let VLMul = m.value, isCommutable = 1 in {
       def "_MM_" # mx : VPseudoBinaryNoMask<VR, VR, VR, "">,
----------------
craig.topper wrote:
> Does this make `PseudoVMANDN` and `PseudoVMORN` commutable? That would be incorrect
Thanks! fixed!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158976



More information about the llvm-commits mailing list