blob: a5a0a98f5a1f39b79699d5880eec5c6780189737 [file] [log] [blame]
Dan Gohmane4aeec02009-10-13 18:30:07 +00001//===- 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"
19using namespace llvm;
20
21// CountCodeReductionForConstant - Figure out an approximation for how many
22// instructions will be constant folded if the specified value is constant.
23//
Dan Gohmane7f0ed52009-10-13 19:58:07 +000024unsigned InlineCostAnalyzer::FunctionInfo::
Dan Gohmane4aeec02009-10-13 18:30:07 +000025 CountCodeReductionForConstant(Value *V) {
26 unsigned Reduction = 0;
27 for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E; ++UI)
28 if (isa<BranchInst>(*UI))
29 Reduction += 40; // Eliminating a conditional branch is a big win
30 else if (SwitchInst *SI = dyn_cast<SwitchInst>(*UI))
31 // Eliminating a switch is a big win, proportional to the number of edges
32 // deleted.
33 Reduction += (SI->getNumSuccessors()-1) * 40;
34 else if (CallInst *CI = dyn_cast<CallInst>(*UI)) {
35 // Turning an indirect call into a direct call is a BIG win
36 Reduction += CI->getCalledValue() == V ? 500 : 0;
37 } else if (InvokeInst *II = dyn_cast<InvokeInst>(*UI)) {
38 // Turning an indirect call into a direct call is a BIG win
39 Reduction += II->getCalledValue() == V ? 500 : 0;
40 } else {
41 // Figure out if this instruction will be removed due to simple constant
42 // propagation.
43 Instruction &Inst = cast<Instruction>(**UI);
44
45 // We can't constant propagate instructions which have effects or
46 // read memory.
47 //
48 // FIXME: It would be nice to capture the fact that a load from a
49 // pointer-to-constant-global is actually a *really* good thing to zap.
50 // Unfortunately, we don't know the pointer that may get propagated here,
51 // so we can't make this decision.
52 if (Inst.mayReadFromMemory() || Inst.mayHaveSideEffects() ||
Victor Hernandez7b929da2009-10-23 21:09:37 +000053 isa<AllocaInst>(Inst))
Dan Gohmane4aeec02009-10-13 18:30:07 +000054 continue;
55
56 bool AllOperandsConstant = true;
57 for (unsigned i = 0, e = Inst.getNumOperands(); i != e; ++i)
58 if (!isa<Constant>(Inst.getOperand(i)) && Inst.getOperand(i) != V) {
59 AllOperandsConstant = false;
60 break;
61 }
62
63 if (AllOperandsConstant) {
64 // We will get to remove this instruction...
65 Reduction += 7;
66
67 // And any other instructions that use it which become constants
68 // themselves.
69 Reduction += CountCodeReductionForConstant(&Inst);
70 }
71 }
72
73 return Reduction;
74}
75
76// CountCodeReductionForAlloca - Figure out an approximation of how much smaller
77// the function will be if it is inlined into a context where an argument
78// becomes an alloca.
79//
Dan Gohmane7f0ed52009-10-13 19:58:07 +000080unsigned InlineCostAnalyzer::FunctionInfo::
Dan Gohmane4aeec02009-10-13 18:30:07 +000081 CountCodeReductionForAlloca(Value *V) {
82 if (!isa<PointerType>(V->getType())) return 0; // Not a pointer
83 unsigned Reduction = 0;
84 for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E;++UI){
85 Instruction *I = cast<Instruction>(*UI);
86 if (isa<LoadInst>(I) || isa<StoreInst>(I))
87 Reduction += 10;
88 else if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(I)) {
89 // If the GEP has variable indices, we won't be able to do much with it.
Jakob Stoklund Olesen026ac3a2010-01-26 21:31:35 +000090 if (GEP->hasAllConstantIndices())
91 Reduction += CountCodeReductionForAlloca(GEP);
Dan Gohmane4aeec02009-10-13 18:30:07 +000092 } else {
93 // If there is some other strange instruction, we're not going to be able
94 // to do much if we inline this.
95 return 0;
96 }
97 }
98
99 return Reduction;
100}
101
Eric Christopher745d8c92010-01-14 21:48:00 +0000102// callIsSmall - If a call is likely to lower to a single target instruction, or
Eric Christopher2d59ae62010-01-14 20:12:34 +0000103// is otherwise deemed small return true.
104// TODO: Perhaps calls like memcpy, strcpy, etc?
105static bool callIsSmall(const Function *F) {
Eric Christopher745d8c92010-01-14 21:48:00 +0000106 if (!F) return false;
107
108 if (F->hasLocalLinkage()) return false;
109
Eric Christopher83c20d32010-01-14 23:00:10 +0000110 if (!F->hasName()) return false;
111
112 StringRef Name = F->getName();
113
114 // These will all likely lower to a single selection DAG node.
115 if (Name == "copysign" || Name == "copysignf" ||
116 Name == "fabs" || Name == "fabsf" || Name == "fabsl" ||
117 Name == "sin" || Name == "sinf" || Name == "sinl" ||
118 Name == "cos" || Name == "cosf" || Name == "cosl" ||
119 Name == "sqrt" || Name == "sqrtf" || Name == "sqrtl" )
120 return true;
121
122 // These are all likely to be optimized into something smaller.
123 if (Name == "pow" || Name == "powf" || Name == "powl" ||
124 Name == "exp2" || Name == "exp2l" || Name == "exp2f" ||
125 Name == "floor" || Name == "floorf" || Name == "ceil" ||
126 Name == "round" || Name == "ffs" || Name == "ffsl" ||
127 Name == "abs" || Name == "labs" || Name == "llabs")
128 return true;
129
Eric Christopher2d59ae62010-01-14 20:12:34 +0000130 return false;
131}
132
Dan Gohmane4aeec02009-10-13 18:30:07 +0000133/// analyzeBasicBlock - Fill in the current structure with information gleaned
134/// from the specified block.
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000135void CodeMetrics::analyzeBasicBlock(const BasicBlock *BB) {
Dan Gohmane4aeec02009-10-13 18:30:07 +0000136 ++NumBlocks;
137
138 for (BasicBlock::const_iterator II = BB->begin(), E = BB->end();
139 II != E; ++II) {
140 if (isa<PHINode>(II)) continue; // PHI nodes don't count.
141
142 // Special handling for calls.
143 if (isa<CallInst>(II) || isa<InvokeInst>(II)) {
144 if (isa<DbgInfoIntrinsic>(II))
145 continue; // Debug intrinsics don't count as size.
146
147 CallSite CS = CallSite::get(const_cast<Instruction*>(&*II));
148
149 // If this function contains a call to setjmp or _setjmp, never inline
150 // it. This is a hack because we depend on the user marking their local
151 // variables as volatile if they are live across a setjmp call, and they
152 // probably won't do this in callers.
153 if (Function *F = CS.getCalledFunction())
154 if (F->isDeclaration() &&
Dan Gohman497f6192009-10-13 20:10:10 +0000155 (F->getName() == "setjmp" || F->getName() == "_setjmp"))
Dan Gohmane4aeec02009-10-13 18:30:07 +0000156 NeverInline = true;
Dan Gohmane4aeec02009-10-13 18:30:07 +0000157
158 // Calls often compile into many machine instructions. Bump up their
159 // cost to reflect this.
Eric Christopher2d59ae62010-01-14 20:12:34 +0000160 if (!isa<IntrinsicInst>(II) && !callIsSmall(CS.getCalledFunction()))
Dan Gohmane4aeec02009-10-13 18:30:07 +0000161 NumInsts += InlineConstants::CallPenalty;
162 }
163
Dan Gohmane4aeec02009-10-13 18:30:07 +0000164 if (const AllocaInst *AI = dyn_cast<AllocaInst>(II)) {
165 if (!AI->isStaticAlloca())
166 this->usesDynamicAlloca = true;
167 }
168
169 if (isa<ExtractElementInst>(II) || isa<VectorType>(II->getType()))
170 ++NumVectorInsts;
171
Dan Gohmane4aeec02009-10-13 18:30:07 +0000172 if (const CastInst *CI = dyn_cast<CastInst>(II)) {
Evan Cheng1a67dd22010-01-14 21:04:31 +0000173 // Noop casts, including ptr <-> int, don't count.
Dan Gohmane4aeec02009-10-13 18:30:07 +0000174 if (CI->isLosslessCast() || isa<IntToPtrInst>(CI) ||
175 isa<PtrToIntInst>(CI))
176 continue;
Evan Cheng1a67dd22010-01-14 21:04:31 +0000177 // Result of a cmp instruction is often extended (to be used by other
178 // cmp instructions, logical or return instructions). These are usually
179 // nop on most sane targets.
180 if (isa<CmpInst>(CI->getOperand(0)))
181 continue;
Chris Lattnerb93a23a2009-11-01 03:07:53 +0000182 } else if (const GetElementPtrInst *GEPI = dyn_cast<GetElementPtrInst>(II)){
Dan Gohmane4aeec02009-10-13 18:30:07 +0000183 // If a GEP has all constant indices, it will probably be folded with
184 // a load/store.
185 if (GEPI->hasAllConstantIndices())
186 continue;
187 }
188
Dan Gohmane4aeec02009-10-13 18:30:07 +0000189 ++NumInsts;
190 }
Chris Lattnerb93a23a2009-11-01 03:07:53 +0000191
192 if (isa<ReturnInst>(BB->getTerminator()))
193 ++NumRets;
194
Chris Lattner66588702009-11-01 18:16:30 +0000195 // We never want to inline functions that contain an indirectbr. This is
Duncan Sandsb0469642009-11-01 19:12:43 +0000196 // incorrect because all the blockaddress's (in static global initializers
197 // for example) would be referring to the original function, and this indirect
Chris Lattner66588702009-11-01 18:16:30 +0000198 // jump would jump from the inlined copy of the function into the original
199 // function which is extremely undefined behavior.
Chris Lattnerb93a23a2009-11-01 03:07:53 +0000200 if (isa<IndirectBrInst>(BB->getTerminator()))
201 NeverInline = true;
Dan Gohmane4aeec02009-10-13 18:30:07 +0000202}
203
204/// analyzeFunction - Fill in the current structure with information gleaned
205/// from the specified function.
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000206void CodeMetrics::analyzeFunction(Function *F) {
207 // Look at the size of the callee.
Dan Gohmane4aeec02009-10-13 18:30:07 +0000208 for (Function::const_iterator BB = F->begin(), E = F->end(); BB != E; ++BB)
209 analyzeBasicBlock(&*BB);
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000210}
211
212/// analyzeFunction - Fill in the current structure with information gleaned
213/// from the specified function.
214void InlineCostAnalyzer::FunctionInfo::analyzeFunction(Function *F) {
215 Metrics.analyzeFunction(F);
Dan Gohmane4aeec02009-10-13 18:30:07 +0000216
217 // A function with exactly one return has it removed during the inlining
218 // process (see InlineFunction), so don't count it.
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000219 // FIXME: This knowledge should really be encoded outside of FunctionInfo.
220 if (Metrics.NumRets==1)
221 --Metrics.NumInsts;
Dan Gohmane4aeec02009-10-13 18:30:07 +0000222
Jakob Stoklund Olesene3039b62010-01-26 21:31:24 +0000223 // Don't bother calculating argument weights if we are never going to inline
224 // the function anyway.
225 if (Metrics.NeverInline)
226 return;
227
Dan Gohmane4aeec02009-10-13 18:30:07 +0000228 // Check out all of the arguments to the function, figuring out how much
229 // code can be eliminated if one of the arguments is a constant.
Jakob Stoklund Olesene3039b62010-01-26 21:31:24 +0000230 ArgumentWeights.reserve(F->arg_size());
Dan Gohmane4aeec02009-10-13 18:30:07 +0000231 for (Function::arg_iterator I = F->arg_begin(), E = F->arg_end(); I != E; ++I)
232 ArgumentWeights.push_back(ArgInfo(CountCodeReductionForConstant(I),
233 CountCodeReductionForAlloca(I)));
234}
235
Dan Gohmane4aeec02009-10-13 18:30:07 +0000236// getInlineCost - The heuristic used to determine if we should inline the
237// function call or not.
238//
239InlineCost InlineCostAnalyzer::getInlineCost(CallSite CS,
240 SmallPtrSet<const Function *, 16> &NeverInline) {
241 Instruction *TheCall = CS.getInstruction();
242 Function *Callee = CS.getCalledFunction();
243 Function *Caller = TheCall->getParent()->getParent();
244
245 // Don't inline functions which can be redefined at link-time to mean
246 // something else. Don't inline functions marked noinline.
247 if (Callee->mayBeOverridden() ||
248 Callee->hasFnAttr(Attribute::NoInline) || NeverInline.count(Callee))
249 return llvm::InlineCost::getNever();
250
251 // InlineCost - This value measures how good of an inline candidate this call
252 // site is to inline. A lower inline cost make is more likely for the call to
253 // be inlined. This value may go negative.
254 //
255 int InlineCost = 0;
256
257 // If there is only one call of the function, and it has internal linkage,
258 // make it almost guaranteed to be inlined.
259 //
260 if (Callee->hasLocalLinkage() && Callee->hasOneUse())
261 InlineCost += InlineConstants::LastCallToStaticBonus;
262
263 // If this function uses the coldcc calling convention, prefer not to inline
264 // it.
265 if (Callee->getCallingConv() == CallingConv::Cold)
266 InlineCost += InlineConstants::ColdccPenalty;
267
268 // If the instruction after the call, or if the normal destination of the
269 // invoke is an unreachable instruction, the function is noreturn. As such,
270 // there is little point in inlining this.
271 if (InvokeInst *II = dyn_cast<InvokeInst>(TheCall)) {
272 if (isa<UnreachableInst>(II->getNormalDest()->begin()))
273 InlineCost += InlineConstants::NoreturnPenalty;
274 } else if (isa<UnreachableInst>(++BasicBlock::iterator(TheCall)))
275 InlineCost += InlineConstants::NoreturnPenalty;
276
277 // Get information about the callee...
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000278 FunctionInfo &CalleeFI = CachedFunctionInfo[Callee];
Dan Gohmane4aeec02009-10-13 18:30:07 +0000279
280 // If we haven't calculated this information yet, do so now.
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000281 if (CalleeFI.Metrics.NumBlocks == 0)
Dan Gohmane4aeec02009-10-13 18:30:07 +0000282 CalleeFI.analyzeFunction(Callee);
283
284 // If we should never inline this, return a huge cost.
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000285 if (CalleeFI.Metrics.NeverInline)
Dan Gohmane4aeec02009-10-13 18:30:07 +0000286 return InlineCost::getNever();
287
288 // FIXME: It would be nice to kill off CalleeFI.NeverInline. Then we
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000289 // could move this up and avoid computing the FunctionInfo for
Dan Gohmane4aeec02009-10-13 18:30:07 +0000290 // things we are going to just return always inline for. This
291 // requires handling setjmp somewhere else, however.
292 if (!Callee->isDeclaration() && Callee->hasFnAttr(Attribute::AlwaysInline))
293 return InlineCost::getAlways();
294
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000295 if (CalleeFI.Metrics.usesDynamicAlloca) {
Dan Gohmane4aeec02009-10-13 18:30:07 +0000296 // Get infomation about the caller...
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000297 FunctionInfo &CallerFI = CachedFunctionInfo[Caller];
Dan Gohmane4aeec02009-10-13 18:30:07 +0000298
299 // If we haven't calculated this information yet, do so now.
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000300 if (CallerFI.Metrics.NumBlocks == 0)
Dan Gohmane4aeec02009-10-13 18:30:07 +0000301 CallerFI.analyzeFunction(Caller);
302
303 // Don't inline a callee with dynamic alloca into a caller without them.
304 // Functions containing dynamic alloca's are inefficient in various ways;
305 // don't create more inefficiency.
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000306 if (!CallerFI.Metrics.usesDynamicAlloca)
Dan Gohmane4aeec02009-10-13 18:30:07 +0000307 return InlineCost::getNever();
308 }
309
310 // Add to the inline quality for properties that make the call valuable to
311 // inline. This includes factors that indicate that the result of inlining
312 // the function will be optimizable. Currently this just looks at arguments
313 // passed into the function.
314 //
315 unsigned ArgNo = 0;
316 for (CallSite::arg_iterator I = CS.arg_begin(), E = CS.arg_end();
317 I != E; ++I, ++ArgNo) {
318 // Each argument passed in has a cost at both the caller and the callee
319 // sides. This favors functions that take many arguments over functions
320 // that take few arguments.
321 InlineCost -= 20;
322
323 // If this is a function being passed in, it is very likely that we will be
324 // able to turn an indirect function call into a direct function call.
325 if (isa<Function>(I))
326 InlineCost -= 100;
327
328 // If an alloca is passed in, inlining this function is likely to allow
329 // significant future optimization possibilities (like scalar promotion, and
330 // scalarization), so encourage the inlining of the function.
331 //
332 else if (isa<AllocaInst>(I)) {
333 if (ArgNo < CalleeFI.ArgumentWeights.size())
334 InlineCost -= CalleeFI.ArgumentWeights[ArgNo].AllocaWeight;
335
336 // If this is a constant being passed into the function, use the argument
337 // weights calculated for the callee to determine how much will be folded
338 // away with this information.
339 } else if (isa<Constant>(I)) {
340 if (ArgNo < CalleeFI.ArgumentWeights.size())
341 InlineCost -= CalleeFI.ArgumentWeights[ArgNo].ConstantWeight;
342 }
343 }
344
345 // Now that we have considered all of the factors that make the call site more
346 // likely to be inlined, look at factors that make us not want to inline it.
347
348 // Don't inline into something too big, which would make it bigger.
349 // "size" here is the number of basic blocks, not instructions.
350 //
351 InlineCost += Caller->size()/15;
352
353 // Look at the size of the callee. Each instruction counts as 5.
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000354 InlineCost += CalleeFI.Metrics.NumInsts*5;
Dan Gohmane4aeec02009-10-13 18:30:07 +0000355
356 return llvm::InlineCost::get(InlineCost);
357}
358
359// getInlineFudgeFactor - Return a > 1.0 factor if the inliner should use a
360// higher threshold to determine if the function call should be inlined.
361float InlineCostAnalyzer::getInlineFudgeFactor(CallSite CS) {
362 Function *Callee = CS.getCalledFunction();
363
364 // Get information about the callee...
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000365 FunctionInfo &CalleeFI = CachedFunctionInfo[Callee];
Dan Gohmane4aeec02009-10-13 18:30:07 +0000366
367 // If we haven't calculated this information yet, do so now.
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000368 if (CalleeFI.Metrics.NumBlocks == 0)
Dan Gohmane4aeec02009-10-13 18:30:07 +0000369 CalleeFI.analyzeFunction(Callee);
370
371 float Factor = 1.0f;
372 // Single BB functions are often written to be inlined.
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000373 if (CalleeFI.Metrics.NumBlocks == 1)
Dan Gohmane4aeec02009-10-13 18:30:07 +0000374 Factor += 0.5f;
375
376 // Be more aggressive if the function contains a good chunk (if it mades up
377 // at least 10% of the instructions) of vector instructions.
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000378 if (CalleeFI.Metrics.NumVectorInsts > CalleeFI.Metrics.NumInsts/2)
Dan Gohmane4aeec02009-10-13 18:30:07 +0000379 Factor += 2.0f;
Dan Gohmane7f0ed52009-10-13 19:58:07 +0000380 else if (CalleeFI.Metrics.NumVectorInsts > CalleeFI.Metrics.NumInsts/10)
Dan Gohmane4aeec02009-10-13 18:30:07 +0000381 Factor += 1.5f;
382 return Factor;
383}