[PATCH] D88890: [AMDGPU] Add patterns for mad/mac legacy f32 instructions

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIInstructions.td:886-887
+def : GCNPat <
+      (f32 (fadd (AMDGPUfmul_legacy (VOP3NoMods f32:$src0),
+                                    (VOP3NoMods f32:$src1)),
+                 (VOP3NoMods f32:$src2))),
----------------
arsenm wrote:
> But these do have modifiers?
Sure, but that's covered by the TODO at line 869. I could repeat the comment here. I assume there was some reason it wasn't totally straightforward (maybe something to do with the tied operands?) else it would've been done already.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88890