[PATCH] D68729: [AMDGPU] Fixed dpp combine of VOP1

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec marked 4 inline comments as done.
rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/GCNDPPCombine.cpp:221-222
       ++NumOperands;
+    } else if (AMDGPU::getNamedOperandIdx(DPPOp,
+                   AMDGPU::OpName::src1_modifiers) != -1) {
+      DPPInst.addImm(0);
----------------
arsenm wrote:
> rampitec wrote:
> > arsenm wrote:
> > > This case isn't tested
> > I do not think such instructions currently exists. VOP2 are tested by the original test.
> Won't this happen for any VOP2 form of an FP instruction? V_MIN_F32_e32 has no modifiers, but V_MIN_F32_dpp does
Good point, thanks!


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

https://reviews.llvm.org/D68729