[PATCH] D123355: [RISCV][NFC] Refactor patterns for Multiply Add instructions

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVVLPatterns.td:1068
-            (!cast<Instruction>("PseudoVWMACC_VV_" # vti.LMul.MX)
-                 wti.RegClass:$rd, vti.RegClass:$rs1, vti.RegClass:$rs2,
-                 GPR:$vl, vti.Log2SEW, TAIL_AGNOSTIC)>;
----------------
frasercrmck wrote:
> We're losing the `wti` in `wti.RegClass:$rd, vti.RegClass:$rs1` here by moving to `VPatWidenMultiplyAddVL_VV_VX`. I'm not sure why this isn't an error.
Doesn't VPatWidenMultiplyAddVL_VV_VX also use `wti.RegClass:$rd, vti.RegClass:$rs1`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123355