[PATCH] D11660: [x86] reassociate integer multiplies using machine combiner pass

[Kevin B. Smith]

kbsmith1 added a comment.

Test LGTM. Slight comment on the code.


================
Comment at: lib/Target/X86/X86InstrInfo.cpp:6312
@@ +6311,3 @@
+  if (Inst.getNumOperands() == 4)
+    if (!Inst.getOperand(3).isDead())
+      return false;
----------------
Is there any possibility of 4 operand instructions where operand number 3 is dead, but is not EFLAGS?  It seems like a check  for (Inst.getOperand(3).isReg && Inst.getOperand(3).getReg() == X86::EFLAGS && !Inst.getOperand(3).isDead()) would be safer and more future proof.


http://reviews.llvm.org/D11660