Separate out the constant bonus from the size reduction metrics. Rework
[oota-llvm.git] / 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 // CountCodeReductionForConstant - Figure out an approximation for how many
146 // instructions will be constant folded if the specified value is constant.
147 //
148 unsigned CodeMetrics::CountCodeReductionForConstant(Value *V) {
149   unsigned Reduction = 0;
150   for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E;++UI){
151     User *U = *UI;
152     if (isa<BranchInst>(U) || isa<SwitchInst>(U)) {
153       // We will be able to eliminate all but one of the successors.
154       const TerminatorInst &TI = cast<TerminatorInst>(*U);
155       const unsigned NumSucc = TI.getNumSuccessors();
156       unsigned Instrs = 0;
157       for (unsigned I = 0; I != NumSucc; ++I)
158         Instrs += NumBBInsts[TI.getSuccessor(I)];
159       // We don't know which blocks will be eliminated, so use the average size.
160       Reduction += InlineConstants::InstrCost*Instrs*(NumSucc-1)/NumSucc;
161     } else {
162       // Figure out if this instruction will be removed due to simple constant
163       // propagation.
164       Instruction &Inst = cast<Instruction>(*U);
165
166       // We can't constant propagate instructions which have effects or
167       // read memory.
168       //
169       // FIXME: It would be nice to capture the fact that a load from a
170       // pointer-to-constant-global is actually a *really* good thing to zap.
171       // Unfortunately, we don't know the pointer that may get propagated here,
172       // so we can't make this decision.
173       if (Inst.mayReadFromMemory() || Inst.mayHaveSideEffects() ||
174           isa<AllocaInst>(Inst))
175         continue;
176
177       bool AllOperandsConstant = true;
178       for (unsigned i = 0, e = Inst.getNumOperands(); i != e; ++i)
179         if (!isa<Constant>(Inst.getOperand(i)) && Inst.getOperand(i) != V) {
180           AllOperandsConstant = false;
181           break;
182         }
183
184       if (AllOperandsConstant) {
185         // We will get to remove this instruction...
186         Reduction += InlineConstants::InstrCost;
187
188         // And any other instructions that use it which become constants
189         // themselves.
190         Reduction += CountCodeReductionForConstant(&Inst);
191       }
192     }
193   }
194   return Reduction;
195 }
196
197 // CountCodeReductionForAlloca - Figure out an approximation of how much smaller
198 // the function will be if it is inlined into a context where an argument
199 // becomes an alloca.
200 //
201 unsigned CodeMetrics::CountCodeReductionForAlloca(Value *V) {
202   if (!V->getType()->isPointerTy()) return 0;  // Not a pointer
203   unsigned Reduction = 0;
204   for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E;++UI){
205     Instruction *I = cast<Instruction>(*UI);
206     if (isa<LoadInst>(I) || isa<StoreInst>(I))
207       Reduction += InlineConstants::InstrCost;
208     else if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(I)) {
209       // If the GEP has variable indices, we won't be able to do much with it.
210       if (GEP->hasAllConstantIndices())
211         Reduction += CountCodeReductionForAlloca(GEP);
212     } else if (BitCastInst *BCI = dyn_cast<BitCastInst>(I)) {
213       // Track pointer through bitcasts.
214       Reduction += CountCodeReductionForAlloca(BCI);
215     } else {
216       // If there is some other strange instruction, we're not going to be able
217       // to do much if we inline this.
218       return 0;
219     }
220   }
221
222   return Reduction;
223 }
224
225 /// analyzeFunction - Fill in the current structure with information gleaned
226 /// from the specified function.
227 void CodeMetrics::analyzeFunction(Function *F) {
228   // Look at the size of the callee.
229   for (Function::const_iterator BB = F->begin(), E = F->end(); BB != E; ++BB)
230     analyzeBasicBlock(&*BB);
231 }
232
233 /// analyzeFunction - Fill in the current structure with information gleaned
234 /// from the specified function.
235 void InlineCostAnalyzer::FunctionInfo::analyzeFunction(Function *F) {
236   Metrics.analyzeFunction(F);
237
238   // A function with exactly one return has it removed during the inlining
239   // process (see InlineFunction), so don't count it.
240   // FIXME: This knowledge should really be encoded outside of FunctionInfo.
241   if (Metrics.NumRets==1)
242     --Metrics.NumInsts;
243
244   // Don't bother calculating argument weights if we are never going to inline
245   // the function anyway.
246   if (NeverInline())
247     return;
248
249   // Check out all of the arguments to the function, figuring out how much
250   // code can be eliminated if one of the arguments is a constant.
251   ArgumentWeights.reserve(F->arg_size());
252   for (Function::arg_iterator I = F->arg_begin(), E = F->arg_end(); I != E; ++I)
253     ArgumentWeights.push_back(ArgInfo(Metrics.CountCodeReductionForConstant(I),
254                                       Metrics.CountCodeReductionForAlloca(I)));
255 }
256
257 /// NeverInline - returns true if the function should never be inlined into
258 /// any caller
259 bool InlineCostAnalyzer::FunctionInfo::NeverInline() {
260   return (Metrics.callsSetJmp || Metrics.isRecursive || 
261           Metrics.containsIndirectBr);
262 }
263 // getSpecializationBonus - The heuristic used to determine the per-call
264 // performance boost for using a specialization of Callee with argument
265 // specializedArgNo replaced by a constant.
266 int InlineCostAnalyzer::getSpecializationBonus(Function *Callee,
267          SmallVectorImpl<unsigned> &SpecializedArgNos)
268 {
269   if (Callee->mayBeOverridden())
270     return 0;
271   
272   int Bonus = 0;
273   // If this function uses the coldcc calling convention, prefer not to
274   // specialize it.
275   if (Callee->getCallingConv() == CallingConv::Cold)
276     Bonus -= InlineConstants::ColdccPenalty;
277   
278   // Get information about the callee.
279   FunctionInfo *CalleeFI = &CachedFunctionInfo[Callee];
280   
281   // If we haven't calculated this information yet, do so now.
282   if (CalleeFI->Metrics.NumBlocks == 0)
283     CalleeFI->analyzeFunction(Callee);
284
285   unsigned ArgNo = 0;
286   unsigned i = 0;
287   for (Function::arg_iterator I = Callee->arg_begin(), E = Callee->arg_end();
288        I != E; ++I, ++ArgNo)
289     if (ArgNo == SpecializedArgNos[i]) {
290       ++i;
291       Bonus += CountBonusForConstant(I);
292     }
293
294   // Calls usually take a long time, so they make the specialization gain 
295   // smaller.
296   Bonus -= CalleeFI->Metrics.NumCalls * InlineConstants::CallPenalty;
297
298   return Bonus;
299 }
300
301 // CountBonusForConstant - Figure out an approximation for how much per-call
302 // performance boost we can expect if the specified value is constant.
303 unsigned InlineCostAnalyzer::CountBonusForConstant(Value *V) {
304   unsigned Bonus = 0;
305   bool indirectCallBonus = false;
306   for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E;++UI){
307     User *U = *UI;
308     if (CallInst *CI = dyn_cast<CallInst>(U)) {
309       // Turning an indirect call into a direct call is a BIG win
310       if (CI->getCalledValue() == V)
311         indirectCallBonus = true;
312     }
313     else if (InvokeInst *II = dyn_cast<InvokeInst>(U)) {
314       // Turning an indirect call into a direct call is a BIG win
315       if (II->getCalledValue() == V)
316         indirectCallBonus = true;
317     }
318     // FIXME: Eliminating conditional branches and switches should
319     // also yield a per-call performance boost.
320     else {
321       // Figure out the bonuses that wll accrue due to simple constant
322       // propagation.
323       Instruction &Inst = cast<Instruction>(*U);
324
325       // We can't constant propagate instructions which have effects or
326       // read memory.
327       //
328       // FIXME: It would be nice to capture the fact that a load from a
329       // pointer-to-constant-global is actually a *really* good thing to zap.
330       // Unfortunately, we don't know the pointer that may get propagated here,
331       // so we can't make this decision.
332       if (Inst.mayReadFromMemory() || Inst.mayHaveSideEffects() ||
333           isa<AllocaInst>(Inst))
334         continue;
335
336       bool AllOperandsConstant = true;
337       for (unsigned i = 0, e = Inst.getNumOperands(); i != e; ++i)
338         if (!isa<Constant>(Inst.getOperand(i)) && Inst.getOperand(i) != V) {
339           AllOperandsConstant = false;
340           break;
341         }
342
343       if (AllOperandsConstant)
344         Bonus += CountBonusForConstant(&Inst);
345     }
346   }
347   
348   // FIXME: The only reason we're applying the bonus once is while it's great
349   // to devirtualize calls the magnitude of the bonus x number of call sites
350   // can lead to a huge code explosion when we prefer to inline 1000 instruction
351   // functions that have 10 call sites. This should be made a function of the
352   // estimated inline penalty/benefit + the indirect call bonus.
353   if (indirectCallBonus) Bonus += InlineConstants::IndirectCallBonus;
354   
355   return Bonus;
356 }
357
358 // getInlineCost - The heuristic used to determine if we should inline the
359 // function call or not.
360 //
361 InlineCost InlineCostAnalyzer::getInlineCost(CallSite CS,
362                                SmallPtrSet<const Function*, 16> &NeverInline) {
363   return getInlineCost(CS, CS.getCalledFunction(), NeverInline);
364 }
365
366 InlineCost InlineCostAnalyzer::getInlineCost(CallSite CS,
367                                Function *Callee,
368                                SmallPtrSet<const Function*, 16> &NeverInline) {
369   Instruction *TheCall = CS.getInstruction();
370   Function *Caller = TheCall->getParent()->getParent();
371   bool isDirectCall = CS.getCalledFunction() == Callee;
372
373   // Don't inline functions which can be redefined at link-time to mean
374   // something else.  Don't inline functions marked noinline or call sites
375   // marked noinline.
376   if (Callee->mayBeOverridden() ||
377       Callee->hasFnAttr(Attribute::NoInline) || NeverInline.count(Callee) ||
378       CS.isNoInline())
379     return llvm::InlineCost::getNever();
380
381   // Get information about the callee.
382   FunctionInfo *CalleeFI = &CachedFunctionInfo[Callee];
383   
384   // If we haven't calculated this information yet, do so now.
385   if (CalleeFI->Metrics.NumBlocks == 0)
386     CalleeFI->analyzeFunction(Callee);
387
388   // If we should never inline this, return a huge cost.
389   if (CalleeFI->NeverInline())
390     return InlineCost::getNever();
391
392   // FIXME: It would be nice to kill off CalleeFI->NeverInline. Then we
393   // could move this up and avoid computing the FunctionInfo for
394   // things we are going to just return always inline for. This
395   // requires handling setjmp somewhere else, however.
396   if (!Callee->isDeclaration() && Callee->hasFnAttr(Attribute::AlwaysInline))
397     return InlineCost::getAlways();
398     
399   if (CalleeFI->Metrics.usesDynamicAlloca) {
400     // Get infomation about the caller.
401     FunctionInfo &CallerFI = CachedFunctionInfo[Caller];
402
403     // If we haven't calculated this information yet, do so now.
404     if (CallerFI.Metrics.NumBlocks == 0) {
405       CallerFI.analyzeFunction(Caller);
406      
407       // Recompute the CalleeFI pointer, getting Caller could have invalidated
408       // it.
409       CalleeFI = &CachedFunctionInfo[Callee];
410     }
411
412     // Don't inline a callee with dynamic alloca into a caller without them.
413     // Functions containing dynamic alloca's are inefficient in various ways;
414     // don't create more inefficiency.
415     if (!CallerFI.Metrics.usesDynamicAlloca)
416       return InlineCost::getNever();
417   }
418
419   // InlineCost - This value measures how good of an inline candidate this call
420   // site is to inline.  A lower inline cost make is more likely for the call to
421   // be inlined.  This value may go negative.
422   //
423   int InlineCost = 0;
424
425   // Add to the inline quality for properties that make the call valuable to
426   // inline.  This includes factors that indicate that the result of inlining
427   // the function will be optimizable.  Currently this just looks at arguments
428   // passed into the function.
429   //
430   unsigned ArgNo = 0;
431   CallSite::arg_iterator I = CS.arg_begin();
432   for (Function::arg_iterator FI = Callee->arg_begin(), FE = Callee->arg_end();
433        FI != FE; ++I, ++FI, ++ArgNo) {
434
435     // If an alloca is passed in, inlining this function is likely to allow
436     // significant future optimization possibilities (like scalar promotion, and
437     // scalarization), so encourage the inlining of the function.
438     //
439     if (isa<AllocaInst>(I))
440       InlineCost -= CalleeFI->ArgumentWeights[ArgNo].AllocaWeight;
441
442     // If this is a constant being passed into the function, use the argument
443     // weights calculated for the callee to determine how much will be folded
444     // away with this information.
445     else if (isa<Constant>(I)) {
446       InlineCost -= CalleeFI->ArgumentWeights[ArgNo].ConstantWeight;
447        
448       // Compute any constant bonus due to inlining we want to give here.
449       InlineCost -= CountBonusForConstant(FI);
450     }
451   }
452   
453   // Each argument passed in has a cost at both the caller and the callee
454   // sides.  Measurements show that each argument costs about the same as an
455   // instruction.
456   InlineCost -= (CS.arg_size() * InlineConstants::InstrCost);
457
458   // If there is only one call of the function, and it has internal linkage,
459   // make it almost guaranteed to be inlined.
460   //
461   if (Callee->hasLocalLinkage() && Callee->hasOneUse() && isDirectCall)
462     InlineCost += InlineConstants::LastCallToStaticBonus;
463
464   // Now that we have considered all of the factors that make the call site more
465   // likely to be inlined, look at factors that make us not want to inline it.
466
467   // If the instruction after the call, or if the normal destination of the
468   // invoke is an unreachable instruction, the function is noreturn.  As such,
469   // there is little point in inlining this.
470   if (InvokeInst *II = dyn_cast<InvokeInst>(TheCall)) {
471     if (isa<UnreachableInst>(II->getNormalDest()->begin()))
472       InlineCost += InlineConstants::NoreturnPenalty;
473   } else if (isa<UnreachableInst>(++BasicBlock::iterator(TheCall)))
474     InlineCost += InlineConstants::NoreturnPenalty;
475   
476   // If this function uses the coldcc calling convention, prefer not to inline
477   // it.
478   if (Callee->getCallingConv() == CallingConv::Cold)
479     InlineCost += InlineConstants::ColdccPenalty;
480
481   // Calls usually take a long time, so they make the inlining gain smaller.
482   InlineCost += CalleeFI->Metrics.NumCalls * InlineConstants::CallPenalty;
483
484   // Look at the size of the callee. Each instruction counts as 5.
485   InlineCost += CalleeFI->Metrics.NumInsts*InlineConstants::InstrCost;
486
487   return llvm::InlineCost::get(InlineCost);
488 }
489
490 // getSpecializationCost - The heuristic used to determine the code-size
491 // impact of creating a specialized version of Callee with argument
492 // SpecializedArgNo replaced by a constant.
493 InlineCost InlineCostAnalyzer::getSpecializationCost(Function *Callee,
494                                SmallVectorImpl<unsigned> &SpecializedArgNos)
495 {
496   // Don't specialize functions which can be redefined at link-time to mean
497   // something else.
498   if (Callee->mayBeOverridden())
499     return llvm::InlineCost::getNever();
500   
501   // Get information about the callee.
502   FunctionInfo *CalleeFI = &CachedFunctionInfo[Callee];
503   
504   // If we haven't calculated this information yet, do so now.
505   if (CalleeFI->Metrics.NumBlocks == 0)
506     CalleeFI->analyzeFunction(Callee);
507
508   int Cost = 0;
509   
510   // Look at the orginal size of the callee.  Each instruction counts as 5.
511   Cost += CalleeFI->Metrics.NumInsts * InlineConstants::InstrCost;
512
513   // Offset that with the amount of code that can be constant-folded
514   // away with the given arguments replaced by constants.
515   for (SmallVectorImpl<unsigned>::iterator an = SpecializedArgNos.begin(),
516        ae = SpecializedArgNos.end(); an != ae; ++an)
517     Cost -= CalleeFI->ArgumentWeights[*an].ConstantWeight;
518
519   return llvm::InlineCost::get(Cost);
520 }
521
522 // getInlineFudgeFactor - Return a > 1.0 factor if the inliner should use a
523 // higher threshold to determine if the function call should be inlined.
524 float InlineCostAnalyzer::getInlineFudgeFactor(CallSite CS) {
525   Function *Callee = CS.getCalledFunction();
526   
527   // Get information about the callee.
528   FunctionInfo &CalleeFI = CachedFunctionInfo[Callee];
529   
530   // If we haven't calculated this information yet, do so now.
531   if (CalleeFI.Metrics.NumBlocks == 0)
532     CalleeFI.analyzeFunction(Callee);
533
534   float Factor = 1.0f;
535   // Single BB functions are often written to be inlined.
536   if (CalleeFI.Metrics.NumBlocks == 1)
537     Factor += 0.5f;
538
539   // Be more aggressive if the function contains a good chunk (if it mades up
540   // at least 10% of the instructions) of vector instructions.
541   if (CalleeFI.Metrics.NumVectorInsts > CalleeFI.Metrics.NumInsts/2)
542     Factor += 2.0f;
543   else if (CalleeFI.Metrics.NumVectorInsts > CalleeFI.Metrics.NumInsts/10)
544     Factor += 1.5f;
545   return Factor;
546 }
547
548 /// growCachedCostInfo - update the cached cost info for Caller after Callee has
549 /// been inlined.
550 void
551 InlineCostAnalyzer::growCachedCostInfo(Function *Caller, Function *Callee) {
552   CodeMetrics &CallerMetrics = CachedFunctionInfo[Caller].Metrics;
553
554   // For small functions we prefer to recalculate the cost for better accuracy.
555   if (CallerMetrics.NumBlocks < 10 || CallerMetrics.NumInsts < 1000) {
556     resetCachedCostInfo(Caller);
557     return;
558   }
559
560   // For large functions, we can save a lot of computation time by skipping
561   // recalculations.
562   if (CallerMetrics.NumCalls > 0)
563     --CallerMetrics.NumCalls;
564
565   if (Callee == 0) return;
566   
567   CodeMetrics &CalleeMetrics = CachedFunctionInfo[Callee].Metrics;
568
569   // If we don't have metrics for the callee, don't recalculate them just to
570   // update an approximation in the caller.  Instead, just recalculate the
571   // caller info from scratch.
572   if (CalleeMetrics.NumBlocks == 0) {
573     resetCachedCostInfo(Caller);
574     return;
575   }
576   
577   // Since CalleeMetrics were already calculated, we know that the CallerMetrics
578   // reference isn't invalidated: both were in the DenseMap.
579   CallerMetrics.usesDynamicAlloca |= CalleeMetrics.usesDynamicAlloca;
580
581   // FIXME: If any of these three are true for the callee, the callee was
582   // not inlined into the caller, so I think they're redundant here.
583   CallerMetrics.callsSetJmp |= CalleeMetrics.callsSetJmp;
584   CallerMetrics.isRecursive |= CalleeMetrics.isRecursive;
585   CallerMetrics.containsIndirectBr |= CalleeMetrics.containsIndirectBr;
586
587   CallerMetrics.NumInsts += CalleeMetrics.NumInsts;
588   CallerMetrics.NumBlocks += CalleeMetrics.NumBlocks;
589   CallerMetrics.NumCalls += CalleeMetrics.NumCalls;
590   CallerMetrics.NumVectorInsts += CalleeMetrics.NumVectorInsts;
591   CallerMetrics.NumRets += CalleeMetrics.NumRets;
592
593   // analyzeBasicBlock counts each function argument as an inst.
594   if (CallerMetrics.NumInsts >= Callee->arg_size())
595     CallerMetrics.NumInsts -= Callee->arg_size();
596   else
597     CallerMetrics.NumInsts = 0;
598   
599   // We are not updating the argument weights. We have already determined that
600   // Caller is a fairly large function, so we accept the loss of precision.
601 }
602
603 /// clear - empty the cache of inline costs
604 void InlineCostAnalyzer::clear() {
605   CachedFunctionInfo.clear();
606 }