[PATCH] D13417: [MachineCombiner] make slack optional in critical path cost calculation (PR25016)

[Sanjay Patel via llvm-commits]

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

Patch updated:

1. Add enum to describe the combiner's objective; this is only 2 items for now, but enables further changes to progress more naturally.
2. Remove check for reduction of instructions; this isn't necessary for the reassociation patterns now that we limit the cost metric for those to depths.


http://reviews.llvm.org/D13417

Files:
  lib/CodeGen/MachineCombiner.cpp
  test/CodeGen/X86/machine-combiner.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13417.39767.patch
Type: text/x-patch
Size: 7948 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151110/506ec626/attachment.bin>