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

[Sanjay Patel via llvm-commits]

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

Patch updated:

1. Fix names of parameters to improvesCriticalPathLen() -> MustReduceCriticalPath, MustReduceDepth
2. For the MustReduceDepth case, we can early exit (don't bother calculating latency for Root and NewRoot).
3. Improve code comments to reflect what we have discussed in this review.


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.39272.patch
Type: text/x-patch
Size: 8333 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151104/660d8823/attachment.bin>