From: Eric Christopher <echristo@apple.com>
Date: Sat, 22 Jan 2011 21:17:33 +0000 (+0000)
Subject: Only apply the devirtualization bonus once instead of per-call site in the
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=7020f12ee8cf8fbf7d5343239c52655352867ccd;p=oota-llvm.git

Only apply the devirtualization bonus once instead of per-call site in the
target function.

Fixes part of rdar://8546196


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@124044 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Analysis/InlineCost.cpp b/lib/Analysis/InlineCost.cpp
index b103897977b..6e251aa2140 100644
--- a/lib/Analysis/InlineCost.cpp
+++ b/lib/Analysis/InlineCost.cpp
@@ -146,17 +146,18 @@ void CodeMetrics::analyzeBasicBlock(const BasicBlock *BB) {
 // performance boost we can expect if the specified value is constant.
 unsigned CodeMetrics::CountBonusForConstant(Value *V) {
   unsigned Bonus = 0;
+  bool indirectCallBonus = false;
   for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E;++UI){
     User *U = *UI;
     if (CallInst *CI = dyn_cast<CallInst>(U)) {
       // Turning an indirect call into a direct call is a BIG win
       if (CI->getCalledValue() == V)
-        Bonus += InlineConstants::IndirectCallBonus;
+        indirectCallBonus = true;
     }
     else if (InvokeInst *II = dyn_cast<InvokeInst>(U)) {
       // Turning an indirect call into a direct call is a BIG win
       if (II->getCalledValue() == V)
-        Bonus += InlineConstants::IndirectCallBonus;
+        indirectCallBonus = true;
     }
     // FIXME: Eliminating conditional branches and switches should
     // also yield a per-call performance boost.
@@ -187,6 +188,9 @@ unsigned CodeMetrics::CountBonusForConstant(Value *V) {
         Bonus += CountBonusForConstant(&Inst);
     }
   }
+  
+  if (indirectCallBonus) Bonus += InlineConstants::IndirectCallBonus;
+  
   return Bonus;
 }