[PATCH] D17584: Cleanup inline cost analysis code

Easwaran Raman via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 13 15:29:45 PDT 2016


eraman updated this revision to Diff 53630.
eraman added a comment.

Remove the parts that are already committed and tweak the remaining part.


http://reviews.llvm.org/D17584

Files:
  lib/Analysis/InlineCost.cpp

Index: lib/Analysis/InlineCost.cpp
===================================================================
--- lib/Analysis/InlineCost.cpp
+++ lib/Analysis/InlineCost.cpp
@@ -1320,7 +1320,7 @@
     // Bail out the moment we cross the threshold. This means we'll under-count
     // the cost, but only when undercounting doesn't matter.
     if (Cost > Threshold)
-      break;
+      return false;
 
     BasicBlock *BB = BBWorklist[Idx];
     if (BB->empty())


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17584.53630.patch
Type: text/x-patch
Size: 457 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160413/2404fcd6/attachment.bin>


More information about the llvm-commits mailing list