lib/Analysis/InlineCost.cpp

   1 //===- InlineCost.cpp - Cost analysis for inliner -------------------------===//
   2 //
   3 //                     The LLVM Compiler Infrastructure
   4 //
   5 // This file is distributed under the University of Illinois Open Source
   6 // License. See LICENSE.TXT for details.
   7 //
   8 //===----------------------------------------------------------------------===//
   9 //
  10 // This file implements inline cost analysis.
  11 //
  12 //===----------------------------------------------------------------------===//
  13
  14 #include "llvm/Analysis/InlineCost.h"
  15 #include "llvm/Support/CallSite.h"
  16 #include "llvm/CallingConv.h"
  17 #include "llvm/IntrinsicInst.h"
  18 #include "llvm/ADT/SmallPtrSet.h"
  19 using namespace llvm;
  20
  21 /// callIsSmall - If a call is likely to lower to a single target instruction,
  22 /// or is otherwise deemed small return true.
  23 /// TODO: Perhaps calls like memcpy, strcpy, etc?
  24 bool llvm::callIsSmall(const Function *F) {
  25   if (!F) return false;
  26
  27   if (F->hasLocalLinkage()) return false;
  28
  29   if (!F->hasName()) return false;
  30
  31   StringRef Name = F->getName();
  32
  33   // These will all likely lower to a single selection DAG node.
  34   if (Name == "copysign" || Name == "copysignf" || Name == "copysignl" ||
  35       Name == "fabs" || Name == "fabsf" || Name == "fabsl" ||
  36       Name == "sin" || Name == "sinf" || Name == "sinl" ||
  37       Name == "cos" || Name == "cosf" || Name == "cosl" ||
  38       Name == "sqrt" || Name == "sqrtf" || Name == "sqrtl" )
  39     return true;
  40
  41   // These are all likely to be optimized into something smaller.
  42   if (Name == "pow" || Name == "powf" || Name == "powl" ||
  43       Name == "exp2" || Name == "exp2l" || Name == "exp2f" ||
  44       Name == "floor" || Name == "floorf" || Name == "ceil" ||
  45       Name == "round" || Name == "ffs" || Name == "ffsl" ||
  46       Name == "abs" || Name == "labs" || Name == "llabs")
  47     return true;
  48
  49   return false;
  50 }
  51
  52 /// analyzeBasicBlock - Fill in the current structure with information gleaned
  53 /// from the specified block.
  54 void CodeMetrics::analyzeBasicBlock(const BasicBlock *BB) {
  55   ++NumBlocks;
  56   unsigned NumInstsBeforeThisBB = NumInsts;
  57   for (BasicBlock::const_iterator II = BB->begin(), E = BB->end();
  58        II != E; ++II) {
  59     if (isa<PHINode>(II)) continue;           // PHI nodes don't count.
  60
  61     // Special handling for calls.
  62     if (isa<CallInst>(II) || isa<InvokeInst>(II)) {
  63       if (isa<DbgInfoIntrinsic>(II))
  64         continue;  // Debug intrinsics don't count as size.
  65
  66       ImmutableCallSite CS(cast<Instruction>(II));
  67
  68       // If this function contains a call to setjmp or _setjmp, never inline
  69       // it.  This is a hack because we depend on the user marking their local
  70       // variables as volatile if they are live across a setjmp call, and they
  71       // probably won't do this in callers.
  72       if (const Function *F = CS.getCalledFunction()) {
  73         // If a function is both internal and has a single use, then it is
  74         // extremely likely to get inlined in the future (it was probably
  75         // exposed by an interleaved devirtualization pass).
  76         if (F->hasInternalLinkage() && F->hasOneUse())
  77           ++NumInlineCandidates;
  78
  79         if (F->isDeclaration() &&
  80             (F->getName() == "setjmp" || F->getName() == "_setjmp"))
  81           callsSetJmp = true;
  82
  83         // If this call is to function itself, then the function is recursive.
  84         // Inlining it into other functions is a bad idea, because this is
  85         // basically just a form of loop peeling, and our metrics aren't useful
  86         // for that case.
  87         if (F == BB->getParent())
  88           isRecursive = true;
  89       }
  90
  91       if (!isa<IntrinsicInst>(II) && !callIsSmall(CS.getCalledFunction())) {
  92         // Each argument to a call takes on average one instruction to set up.
  93         NumInsts += CS.arg_size();
  94
  95         // We don't want inline asm to count as a call - that would prevent loop
  96         // unrolling. The argument setup cost is still real, though.
  97         if (!isa<InlineAsm>(CS.getCalledValue()))
  98           ++NumCalls;
  99       }
 100     }
 101
 102     if (const AllocaInst *AI = dyn_cast<AllocaInst>(II)) {
 103       if (!AI->isStaticAlloca())
 104         this->usesDynamicAlloca = true;
 105     }
 106
 107     if (isa<ExtractElementInst>(II) || II->getType()->isVectorTy())
 108       ++NumVectorInsts;
 109
 110     if (const CastInst *CI = dyn_cast<CastInst>(II)) {
 111       // Noop casts, including ptr <-> int,  don't count.
 112       if (CI->isLosslessCast() || isa<IntToPtrInst>(CI) ||
 113           isa<PtrToIntInst>(CI))
 114         continue;
 115       // Result of a cmp instruction is often extended (to be used by other
 116       // cmp instructions, logical or return instructions). These are usually
 117       // nop on most sane targets.
 118       if (isa<CmpInst>(CI->getOperand(0)))
 119         continue;
 120     } else if (const GetElementPtrInst *GEPI = dyn_cast<GetElementPtrInst>(II)){
 121       // If a GEP has all constant indices, it will probably be folded with
 122       // a load/store.
 123       if (GEPI->hasAllConstantIndices())
 124         continue;
 125     }
 126
 127     ++NumInsts;
 128   }
 129
 130   if (isa<ReturnInst>(BB->getTerminator()))
 131     ++NumRets;
 132
 133   // We never want to inline functions that contain an indirectbr.  This is
 134   // incorrect because all the blockaddress's (in static global initializers
 135   // for example) would be referring to the original function, and this indirect
 136   // jump would jump from the inlined copy of the function into the original
 137   // function which is extremely undefined behavior.
 138   if (isa<IndirectBrInst>(BB->getTerminator()))
 139     containsIndirectBr = true;
 140
 141   // Remember NumInsts for this BB.
 142   NumBBInsts[BB] = NumInsts - NumInstsBeforeThisBB;
 143 }
 144
 145 // CountBonusForConstant - Figure out an approximation for how much per-call
 146 // performance boost we can expect if the specified value is constant.
 147 unsigned CodeMetrics::CountBonusForConstant(Value *V) {
 148   unsigned Bonus = 0;
 149   bool indirectCallBonus = false;
 150   for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E;++UI){
 151     User *U = *UI;
 152     if (CallInst *CI = dyn_cast<CallInst>(U)) {
 153       // Turning an indirect call into a direct call is a BIG win
 154       if (CI->getCalledValue() == V)
 155         indirectCallBonus = true;
 156     }
 157     else if (InvokeInst *II = dyn_cast<InvokeInst>(U)) {
 158       // Turning an indirect call into a direct call is a BIG win
 159       if (II->getCalledValue() == V)
 160         indirectCallBonus = true;
 161     }
 162     // FIXME: Eliminating conditional branches and switches should
 163     // also yield a per-call performance boost.
 164     else {
 165       // Figure out the bonuses that wll accrue due to simple constant
 166       // propagation.
 167       Instruction &Inst = cast<Instruction>(*U);
 168
 169       // We can't constant propagate instructions which have effects or
 170       // read memory.
 171       //
 172       // FIXME: It would be nice to capture the fact that a load from a
 173       // pointer-to-constant-global is actually a *really* good thing to zap.
 174       // Unfortunately, we don't know the pointer that may get propagated here,
 175       // so we can't make this decision.
 176       if (Inst.mayReadFromMemory() || Inst.mayHaveSideEffects() ||
 177           isa<AllocaInst>(Inst))
 178         continue;
 179
 180       bool AllOperandsConstant = true;
 181       for (unsigned i = 0, e = Inst.getNumOperands(); i != e; ++i)
 182         if (!isa<Constant>(Inst.getOperand(i)) && Inst.getOperand(i) != V) {
 183           AllOperandsConstant = false;
 184           break;
 185         }
 186
 187       if (AllOperandsConstant)
 188         Bonus += CountBonusForConstant(&Inst);
 189     }
 190   }
 191
 192   if (indirectCallBonus) Bonus += InlineConstants::IndirectCallBonus;
 193
 194   return Bonus;
 195 }
 196
 197
 198 // CountCodeReductionForConstant - Figure out an approximation for how many
 199 // instructions will be constant folded if the specified value is constant.
 200 //
 201 unsigned CodeMetrics::CountCodeReductionForConstant(Value *V) {
 202   unsigned Reduction = 0;
 203   for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E;++UI){
 204     User *U = *UI;
 205     if (isa<BranchInst>(U) || isa<SwitchInst>(U)) {
 206       // We will be able to eliminate all but one of the successors.
 207       const TerminatorInst &TI = cast<TerminatorInst>(*U);
 208       const unsigned NumSucc = TI.getNumSuccessors();
 209       unsigned Instrs = 0;
 210       for (unsigned I = 0; I != NumSucc; ++I)
 211         Instrs += NumBBInsts[TI.getSuccessor(I)];
 212       // We don't know which blocks will be eliminated, so use the average size.
 213       Reduction += InlineConstants::InstrCost*Instrs*(NumSucc-1)/NumSucc;
 214     } else {
 215       // Figure out if this instruction will be removed due to simple constant
 216       // propagation.
 217       Instruction &Inst = cast<Instruction>(*U);
 218
 219       // We can't constant propagate instructions which have effects or
 220       // read memory.
 221       //
 222       // FIXME: It would be nice to capture the fact that a load from a
 223       // pointer-to-constant-global is actually a *really* good thing to zap.
 224       // Unfortunately, we don't know the pointer that may get propagated here,
 225       // so we can't make this decision.
 226       if (Inst.mayReadFromMemory() || Inst.mayHaveSideEffects() ||
 227           isa<AllocaInst>(Inst))
 228         continue;
 229
 230       bool AllOperandsConstant = true;
 231       for (unsigned i = 0, e = Inst.getNumOperands(); i != e; ++i)
 232         if (!isa<Constant>(Inst.getOperand(i)) && Inst.getOperand(i) != V) {
 233           AllOperandsConstant = false;
 234           break;
 235         }
 236
 237       if (AllOperandsConstant) {
 238         // We will get to remove this instruction...
 239         Reduction += InlineConstants::InstrCost;
 240
 241         // And any other instructions that use it which become constants
 242         // themselves.
 243         Reduction += CountCodeReductionForConstant(&Inst);
 244       }
 245     }
 246   }
 247   return Reduction;
 248 }
 249
 250 // CountCodeReductionForAlloca - Figure out an approximation of how much smaller
 251 // the function will be if it is inlined into a context where an argument
 252 // becomes an alloca.
 253 //
 254 unsigned CodeMetrics::CountCodeReductionForAlloca(Value *V) {
 255   if (!V->getType()->isPointerTy()) return 0;  // Not a pointer
 256   unsigned Reduction = 0;
 257   for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E;++UI){
 258     Instruction *I = cast<Instruction>(*UI);
 259     if (isa<LoadInst>(I) || isa<StoreInst>(I))
 260       Reduction += InlineConstants::InstrCost;
 261     else if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(I)) {
 262       // If the GEP has variable indices, we won't be able to do much with it.
 263       if (GEP->hasAllConstantIndices())
 264         Reduction += CountCodeReductionForAlloca(GEP);
 265     } else if (BitCastInst *BCI = dyn_cast<BitCastInst>(I)) {
 266       // Track pointer through bitcasts.
 267       Reduction += CountCodeReductionForAlloca(BCI);
 268     } else {
 269       // If there is some other strange instruction, we're not going to be able
 270       // to do much if we inline this.
 271       return 0;
 272     }
 273   }
 274
 275   return Reduction;
 276 }
 277
 278 /// analyzeFunction - Fill in the current structure with information gleaned
 279 /// from the specified function.
 280 void CodeMetrics::analyzeFunction(Function *F) {
 281   // Look at the size of the callee.
 282   for (Function::const_iterator BB = F->begin(), E = F->end(); BB != E; ++BB)
 283     analyzeBasicBlock(&*BB);
 284 }
 285
 286 /// analyzeFunction - Fill in the current structure with information gleaned
 287 /// from the specified function.
 288 void InlineCostAnalyzer::FunctionInfo::analyzeFunction(Function *F) {
 289   Metrics.analyzeFunction(F);
 290
 291   // A function with exactly one return has it removed during the inlining
 292   // process (see InlineFunction), so don't count it.
 293   // FIXME: This knowledge should really be encoded outside of FunctionInfo.
 294   if (Metrics.NumRets==1)
 295     --Metrics.NumInsts;
 296
 297   // Don't bother calculating argument weights if we are never going to inline
 298   // the function anyway.
 299   if (NeverInline())
 300     return;
 301
 302   // Check out all of the arguments to the function, figuring out how much
 303   // code can be eliminated if one of the arguments is a constant.
 304   ArgumentWeights.reserve(F->arg_size());
 305   for (Function::arg_iterator I = F->arg_begin(), E = F->arg_end(); I != E; ++I)
 306     ArgumentWeights.push_back(ArgInfo(Metrics.CountCodeReductionForConstant(I),
 307                                       Metrics.CountCodeReductionForAlloca(I),
 308                                       Metrics.CountBonusForConstant(I)));
 309 }
 310
 311 /// NeverInline - returns true if the function should never be inlined into
 312 /// any caller
 313 bool InlineCostAnalyzer::FunctionInfo::NeverInline()
 314 {
 315   return (Metrics.callsSetJmp || Metrics.isRecursive ||
 316           Metrics.containsIndirectBr);
 317
 318 }
 319 // getSpecializationBonus - The heuristic used to determine the per-call
 320 // performance boost for using a specialization of Callee with argument
 321 // specializedArgNo replaced by a constant.
 322 int InlineCostAnalyzer::getSpecializationBonus(Function *Callee,
 323          SmallVectorImpl<unsigned> &SpecializedArgNos)
 324 {
 325   if (Callee->mayBeOverridden())
 326     return 0;
 327
 328   int Bonus = 0;
 329   // If this function uses the coldcc calling convention, prefer not to
 330   // specialize it.
 331   if (Callee->getCallingConv() == CallingConv::Cold)
 332     Bonus -= InlineConstants::ColdccPenalty;
 333
 334   // Get information about the callee.
 335   FunctionInfo *CalleeFI = &CachedFunctionInfo[Callee];
 336
 337   // If we haven't calculated this information yet, do so now.
 338   if (CalleeFI->Metrics.NumBlocks == 0)
 339     CalleeFI->analyzeFunction(Callee);
 340
 341
 342   for (unsigned i = 0, s = SpecializedArgNos.size();
 343        i < s; ++i )
 344   {
 345     Bonus += CalleeFI->ArgumentWeights[SpecializedArgNos[i]].ConstantBonus;
 346   }
 347   // Calls usually take a long time, so they make the specialization gain
 348   // smaller.
 349   Bonus -= CalleeFI->Metrics.NumCalls * InlineConstants::CallPenalty;
 350
 351   return Bonus;
 352 }
 353
 354
 355 // getInlineCost - The heuristic used to determine if we should inline the
 356 // function call or not.
 357 //
 358 InlineCost InlineCostAnalyzer::getInlineCost(CallSite CS,
 359                                SmallPtrSet<const Function*, 16> &NeverInline) {
 360   return getInlineCost(CS, CS.getCalledFunction(), NeverInline);
 361 }
 362
 363 InlineCost InlineCostAnalyzer::getInlineCost(CallSite CS,
 364                                Function *Callee,
 365                                SmallPtrSet<const Function*, 16> &NeverInline) {
 366   Instruction *TheCall = CS.getInstruction();
 367   Function *Caller = TheCall->getParent()->getParent();
 368   bool isDirectCall = CS.getCalledFunction() == Callee;
 369
 370   // Don't inline functions which can be redefined at link-time to mean
 371   // something else.  Don't inline functions marked noinline or call sites
 372   // marked noinline.
 373   if (Callee->mayBeOverridden() ||
 374       Callee->hasFnAttr(Attribute::NoInline) || NeverInline.count(Callee) ||
 375       CS.isNoInline())
 376     return llvm::InlineCost::getNever();
 377
 378   // InlineCost - This value measures how good of an inline candidate this call
 379   // site is to inline.  A lower inline cost make is more likely for the call to
 380   // be inlined.  This value may go negative.
 381   //
 382   int InlineCost = 0;
 383
 384   // If there is only one call of the function, and it has internal linkage,
 385   // make it almost guaranteed to be inlined.
 386   //
 387   if (Callee->hasLocalLinkage() && Callee->hasOneUse() && isDirectCall)
 388     InlineCost += InlineConstants::LastCallToStaticBonus;
 389
 390   // If this function uses the coldcc calling convention, prefer not to inline
 391   // it.
 392   if (Callee->getCallingConv() == CallingConv::Cold)
 393     InlineCost += InlineConstants::ColdccPenalty;
 394
 395   // If the instruction after the call, or if the normal destination of the
 396   // invoke is an unreachable instruction, the function is noreturn.  As such,
 397   // there is little point in inlining this.
 398   if (InvokeInst *II = dyn_cast<InvokeInst>(TheCall)) {
 399     if (isa<UnreachableInst>(II->getNormalDest()->begin()))
 400       InlineCost += InlineConstants::NoreturnPenalty;
 401   } else if (isa<UnreachableInst>(++BasicBlock::iterator(TheCall)))
 402     InlineCost += InlineConstants::NoreturnPenalty;
 403
 404   // Get information about the callee.
 405   FunctionInfo *CalleeFI = &CachedFunctionInfo[Callee];
 406
 407   // If we haven't calculated this information yet, do so now.
 408   if (CalleeFI->Metrics.NumBlocks == 0)
 409     CalleeFI->analyzeFunction(Callee);
 410
 411   // If we should never inline this, return a huge cost.
 412   if (CalleeFI->NeverInline())
 413     return InlineCost::getNever();
 414
 415   // FIXME: It would be nice to kill off CalleeFI->NeverInline. Then we
 416   // could move this up and avoid computing the FunctionInfo for
 417   // things we are going to just return always inline for. This
 418   // requires handling setjmp somewhere else, however.
 419   if (!Callee->isDeclaration() && Callee->hasFnAttr(Attribute::AlwaysInline))
 420     return InlineCost::getAlways();
 421
 422   if (CalleeFI->Metrics.usesDynamicAlloca) {
 423     // Get infomation about the caller.
 424     FunctionInfo &CallerFI = CachedFunctionInfo[Caller];
 425
 426     // If we haven't calculated this information yet, do so now.
 427     if (CallerFI.Metrics.NumBlocks == 0) {
 428       CallerFI.analyzeFunction(Caller);
 429
 430       // Recompute the CalleeFI pointer, getting Caller could have invalidated
 431       // it.
 432       CalleeFI = &CachedFunctionInfo[Callee];
 433     }
 434
 435     // Don't inline a callee with dynamic alloca into a caller without them.
 436     // Functions containing dynamic alloca's are inefficient in various ways;
 437     // don't create more inefficiency.
 438     if (!CallerFI.Metrics.usesDynamicAlloca)
 439       return InlineCost::getNever();
 440   }
 441
 442   // Add to the inline quality for properties that make the call valuable to
 443   // inline.  This includes factors that indicate that the result of inlining
 444   // the function will be optimizable.  Currently this just looks at arguments
 445   // passed into the function.
 446   //
 447   unsigned ArgNo = 0;
 448   for (CallSite::arg_iterator I = CS.arg_begin(), E = CS.arg_end();
 449        I != E; ++I, ++ArgNo) {
 450     // Each argument passed in has a cost at both the caller and the callee
 451     // sides.  Measurements show that each argument costs about the same as an
 452     // instruction.
 453     InlineCost -= InlineConstants::InstrCost;
 454
 455     // If an alloca is passed in, inlining this function is likely to allow
 456     // significant future optimization possibilities (like scalar promotion, and
 457     // scalarization), so encourage the inlining of the function.
 458     //
 459     if (isa<AllocaInst>(I)) {
 460       if (ArgNo < CalleeFI->ArgumentWeights.size())
 461         InlineCost -= CalleeFI->ArgumentWeights[ArgNo].AllocaWeight;
 462
 463       // If this is a constant being passed into the function, use the argument
 464       // weights calculated for the callee to determine how much will be folded
 465       // away with this information.
 466     } else if (isa<Constant>(I)) {
 467       if (ArgNo < CalleeFI->ArgumentWeights.size())
 468         InlineCost -= (CalleeFI->ArgumentWeights[ArgNo].ConstantWeight +
 469                        CalleeFI->ArgumentWeights[ArgNo].ConstantBonus);
 470     }
 471   }
 472
 473   // Now that we have considered all of the factors that make the call site more
 474   // likely to be inlined, look at factors that make us not want to inline it.
 475
 476   // Calls usually take a long time, so they make the inlining gain smaller.
 477   InlineCost += CalleeFI->Metrics.NumCalls * InlineConstants::CallPenalty;
 478
 479   // Look at the size of the callee. Each instruction counts as 5.
 480   InlineCost += CalleeFI->Metrics.NumInsts*InlineConstants::InstrCost;
 481
 482   return llvm::InlineCost::get(InlineCost);
 483 }
 484
 485 // getSpecializationCost - The heuristic used to determine the code-size
 486 // impact of creating a specialized version of Callee with argument
 487 // SpecializedArgNo replaced by a constant.
 488 InlineCost InlineCostAnalyzer::getSpecializationCost(Function *Callee,
 489                                SmallVectorImpl<unsigned> &SpecializedArgNos)
 490 {
 491   // Don't specialize functions which can be redefined at link-time to mean
 492   // something else.
 493   if (Callee->mayBeOverridden())
 494     return llvm::InlineCost::getNever();
 495
 496   // Get information about the callee.
 497   FunctionInfo *CalleeFI = &CachedFunctionInfo[Callee];
 498
 499   // If we haven't calculated this information yet, do so now.
 500   if (CalleeFI->Metrics.NumBlocks == 0)
 501     CalleeFI->analyzeFunction(Callee);
 502
 503   int Cost = 0;
 504
 505   // Look at the orginal size of the callee.  Each instruction counts as 5.
 506   Cost += CalleeFI->Metrics.NumInsts * InlineConstants::InstrCost;
 507
 508   // Offset that with the amount of code that can be constant-folded
 509   // away with the given arguments replaced by constants.
 510   for (SmallVectorImpl<unsigned>::iterator an = SpecializedArgNos.begin(),
 511        ae = SpecializedArgNos.end(); an != ae; ++an)
 512   {
 513     Cost -= CalleeFI->ArgumentWeights[*an].ConstantWeight;
 514   }
 515
 516   return llvm::InlineCost::get(Cost);
 517 }
 518
 519 // getInlineFudgeFactor - Return a > 1.0 factor if the inliner should use a
 520 // higher threshold to determine if the function call should be inlined.
 521 float InlineCostAnalyzer::getInlineFudgeFactor(CallSite CS) {
 522   Function *Callee = CS.getCalledFunction();
 523
 524   // Get information about the callee.
 525   FunctionInfo &CalleeFI = CachedFunctionInfo[Callee];
 526
 527   // If we haven't calculated this information yet, do so now.
 528   if (CalleeFI.Metrics.NumBlocks == 0)
 529     CalleeFI.analyzeFunction(Callee);
 530
 531   float Factor = 1.0f;
 532   // Single BB functions are often written to be inlined.
 533   if (CalleeFI.Metrics.NumBlocks == 1)
 534     Factor += 0.5f;
 535
 536   // Be more aggressive if the function contains a good chunk (if it mades up
 537   // at least 10% of the instructions) of vector instructions.
 538   if (CalleeFI.Metrics.NumVectorInsts > CalleeFI.Metrics.NumInsts/2)
 539     Factor += 2.0f;
 540   else if (CalleeFI.Metrics.NumVectorInsts > CalleeFI.Metrics.NumInsts/10)
 541     Factor += 1.5f;
 542   return Factor;
 543 }
 544
 545 /// growCachedCostInfo - update the cached cost info for Caller after Callee has
 546 /// been inlined.
 547 void
 548 InlineCostAnalyzer::growCachedCostInfo(Function *Caller, Function *Callee) {
 549   CodeMetrics &CallerMetrics = CachedFunctionInfo[Caller].Metrics;
 550
 551   // For small functions we prefer to recalculate the cost for better accuracy.
 552   if (CallerMetrics.NumBlocks < 10 || CallerMetrics.NumInsts < 1000) {
 553     resetCachedCostInfo(Caller);
 554     return;
 555   }
 556
 557   // For large functions, we can save a lot of computation time by skipping
 558   // recalculations.
 559   if (CallerMetrics.NumCalls > 0)
 560     --CallerMetrics.NumCalls;
 561
 562   if (Callee == 0) return;
 563
 564   CodeMetrics &CalleeMetrics = CachedFunctionInfo[Callee].Metrics;
 565
 566   // If we don't have metrics for the callee, don't recalculate them just to
 567   // update an approximation in the caller.  Instead, just recalculate the
 568   // caller info from scratch.
 569   if (CalleeMetrics.NumBlocks == 0) {
 570     resetCachedCostInfo(Caller);
 571     return;
 572   }
 573
 574   // Since CalleeMetrics were already calculated, we know that the CallerMetrics
 575   // reference isn't invalidated: both were in the DenseMap.
 576   CallerMetrics.usesDynamicAlloca |= CalleeMetrics.usesDynamicAlloca;
 577
 578   // FIXME: If any of these three are true for the callee, the callee was
 579   // not inlined into the caller, so I think they're redundant here.
 580   CallerMetrics.callsSetJmp |= CalleeMetrics.callsSetJmp;
 581   CallerMetrics.isRecursive |= CalleeMetrics.isRecursive;
 582   CallerMetrics.containsIndirectBr |= CalleeMetrics.containsIndirectBr;
 583
 584   CallerMetrics.NumInsts += CalleeMetrics.NumInsts;
 585   CallerMetrics.NumBlocks += CalleeMetrics.NumBlocks;
 586   CallerMetrics.NumCalls += CalleeMetrics.NumCalls;
 587   CallerMetrics.NumVectorInsts += CalleeMetrics.NumVectorInsts;
 588   CallerMetrics.NumRets += CalleeMetrics.NumRets;
 589
 590   // analyzeBasicBlock counts each function argument as an inst.
 591   if (CallerMetrics.NumInsts >= Callee->arg_size())
 592     CallerMetrics.NumInsts -= Callee->arg_size();
 593   else
 594     CallerMetrics.NumInsts = 0;
 595
 596   // We are not updating the argument weights. We have already determined that
 597   // Caller is a fairly large function, so we accept the loss of precision.
 598 }
 599
 600 /// clear - empty the cache of inline costs
 601 void InlineCostAnalyzer::clear() {
 602   CachedFunctionInfo.clear();
 603 }