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

[Sanjay Patel via llvm-commits]

spatel retitled this revision from "[MachineCombiner] remove "slack"  from critical path cost calculation (PR25016)" to "[MachineCombiner] make slack optional in critical path cost calculation (PR25016)".
spatel updated the summary for this revision.
spatel updated this revision to Diff 36552.
spatel added a comment.
Herald added a subscriber: aemerson.

Patch updated:
Make slack an optional part of the critical path calculation. Slack is still used with the AArch64 MULADD patterns, but not with the reassociation patterns where it may cause harm.


http://reviews.llvm.org/D13417

Files:
  include/llvm/CodeGen/MachineCombinerPattern.h
  lib/CodeGen/MachineCombiner.cpp
  test/CodeGen/X86/machine-combiner.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13417.36552.patch
Type: text/x-patch
Size: 4889 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151005/3dcf4173/attachment.bin>