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(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(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; }