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

Sanjay Patel spatel at rotateright.com
Thu Jul 30 16:50:55 PDT 2015


spatel updated this revision to Diff 31089.
spatel added a comment.

Patch updated based on feedback from Gerolf and Kevin (thanks!):

1. Rename hasVirtualRegDefsInBasicBlock -> hasReassociableOperands  (the function is more general now)
2. Update comment on EFLAGS check to make it clearer (I hope)
3. Add asserts about that 3rd operand which we assume must be EFLAGS


http://reviews.llvm.org/D11660

Files:
  lib/Target/X86/X86InstrInfo.cpp
  test/CodeGen/X86/machine-combiner-int.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11660.31089.patch
Type: text/x-patch
Size: 4511 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150730/9912fe57/attachment.bin>


More information about the llvm-commits mailing list