Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 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" |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 18 | #include "llvm/Target/TargetData.h" |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 19 | #include "llvm/ADT/SmallPtrSet.h" |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 20 | |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 21 | using namespace llvm; |
| 22 | |
Dan Gohman | 52d55bd | 2010-04-16 15:14:50 +0000 | [diff] [blame] | 23 | /// callIsSmall - If a call is likely to lower to a single target instruction, |
| 24 | /// or is otherwise deemed small return true. |
| 25 | /// TODO: Perhaps calls like memcpy, strcpy, etc? |
| 26 | bool llvm::callIsSmall(const Function *F) { |
Eric Christopher | 745d8c9 | 2010-01-14 21:48:00 +0000 | [diff] [blame] | 27 | if (!F) return false; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 28 | |
Eric Christopher | 745d8c9 | 2010-01-14 21:48:00 +0000 | [diff] [blame] | 29 | if (F->hasLocalLinkage()) return false; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 30 | |
Eric Christopher | 83c20d3 | 2010-01-14 23:00:10 +0000 | [diff] [blame] | 31 | if (!F->hasName()) return false; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 32 | |
Eric Christopher | 83c20d3 | 2010-01-14 23:00:10 +0000 | [diff] [blame] | 33 | StringRef Name = F->getName(); |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 34 | |
Eric Christopher | 83c20d3 | 2010-01-14 23:00:10 +0000 | [diff] [blame] | 35 | // These will all likely lower to a single selection DAG node. |
Duncan Sands | 87cd7ed | 2010-03-15 14:01:44 +0000 | [diff] [blame] | 36 | if (Name == "copysign" || Name == "copysignf" || Name == "copysignl" || |
Eric Christopher | 83c20d3 | 2010-01-14 23:00:10 +0000 | [diff] [blame] | 37 | Name == "fabs" || Name == "fabsf" || Name == "fabsl" || |
| 38 | Name == "sin" || Name == "sinf" || Name == "sinl" || |
| 39 | Name == "cos" || Name == "cosf" || Name == "cosl" || |
| 40 | Name == "sqrt" || Name == "sqrtf" || Name == "sqrtl" ) |
| 41 | return true; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 42 | |
Eric Christopher | 83c20d3 | 2010-01-14 23:00:10 +0000 | [diff] [blame] | 43 | // These are all likely to be optimized into something smaller. |
| 44 | if (Name == "pow" || Name == "powf" || Name == "powl" || |
| 45 | Name == "exp2" || Name == "exp2l" || Name == "exp2f" || |
| 46 | Name == "floor" || Name == "floorf" || Name == "ceil" || |
| 47 | Name == "round" || Name == "ffs" || Name == "ffsl" || |
| 48 | Name == "abs" || Name == "labs" || Name == "llabs") |
| 49 | return true; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 50 | |
Eric Christopher | 2d59ae6 | 2010-01-14 20:12:34 +0000 | [diff] [blame] | 51 | return false; |
| 52 | } |
| 53 | |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 54 | /// analyzeBasicBlock - Fill in the current structure with information gleaned |
| 55 | /// from the specified block. |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 56 | void CodeMetrics::analyzeBasicBlock(const BasicBlock *BB, |
| 57 | const TargetData *TD) { |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 58 | ++NumBlocks; |
Devang Patel | afc33fa | 2010-03-13 01:05:02 +0000 | [diff] [blame] | 59 | unsigned NumInstsBeforeThisBB = NumInsts; |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 60 | for (BasicBlock::const_iterator II = BB->begin(), E = BB->end(); |
| 61 | II != E; ++II) { |
| 62 | if (isa<PHINode>(II)) continue; // PHI nodes don't count. |
| 63 | |
| 64 | // Special handling for calls. |
| 65 | if (isa<CallInst>(II) || isa<InvokeInst>(II)) { |
Nick Lewycky | 6c6fcc4 | 2011-12-21 20:26:03 +0000 | [diff] [blame] | 66 | if (const IntrinsicInst *IntrinsicI = dyn_cast<IntrinsicInst>(II)) { |
| 67 | switch (IntrinsicI->getIntrinsicID()) { |
| 68 | default: break; |
| 69 | case Intrinsic::dbg_declare: |
| 70 | case Intrinsic::dbg_value: |
| 71 | case Intrinsic::invariant_start: |
| 72 | case Intrinsic::invariant_end: |
| 73 | case Intrinsic::lifetime_start: |
| 74 | case Intrinsic::lifetime_end: |
| 75 | case Intrinsic::objectsize: |
| 76 | case Intrinsic::ptr_annotation: |
| 77 | case Intrinsic::var_annotation: |
| 78 | // These intrinsics don't count as size. |
| 79 | continue; |
| 80 | } |
| 81 | } |
Gabor Greif | 0de11e0 | 2010-07-27 14:15:29 +0000 | [diff] [blame] | 82 | |
| 83 | ImmutableCallSite CS(cast<Instruction>(II)); |
| 84 | |
Gabor Greif | 0de11e0 | 2010-07-27 14:15:29 +0000 | [diff] [blame] | 85 | if (const Function *F = CS.getCalledFunction()) { |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 86 | // If a function is both internal and has a single use, then it is |
| 87 | // extremely likely to get inlined in the future (it was probably |
Owen Anderson | f9a26b8 | 2010-09-09 20:32:23 +0000 | [diff] [blame] | 88 | // exposed by an interleaved devirtualization pass). |
Nick Lewycky | 20aded5 | 2011-12-21 06:06:30 +0000 | [diff] [blame] | 89 | if (!CS.isNoInline() && F->hasInternalLinkage() && F->hasOneUse()) |
Owen Anderson | f9a26b8 | 2010-09-09 20:32:23 +0000 | [diff] [blame] | 90 | ++NumInlineCandidates; |
Rafael Espindola | ac53b0a | 2011-05-16 15:48:45 +0000 | [diff] [blame] | 91 | |
Chris Lattner | 4b7b42c | 2010-04-30 22:37:22 +0000 | [diff] [blame] | 92 | // If this call is to function itself, then the function is recursive. |
| 93 | // Inlining it into other functions is a bad idea, because this is |
| 94 | // basically just a form of loop peeling, and our metrics aren't useful |
| 95 | // for that case. |
| 96 | if (F == BB->getParent()) |
Kenneth Uildriks | 42c7d23 | 2010-06-09 15:11:37 +0000 | [diff] [blame] | 97 | isRecursive = true; |
Chris Lattner | 4b7b42c | 2010-04-30 22:37:22 +0000 | [diff] [blame] | 98 | } |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 99 | |
Nick Lewycky | 6c6fcc4 | 2011-12-21 20:26:03 +0000 | [diff] [blame] | 100 | if (!isa<IntrinsicInst>(II) && !callIsSmall(CS.getCalledFunction())) { |
Jakob Stoklund Olesen | aa034fa | 2010-02-05 23:21:18 +0000 | [diff] [blame] | 101 | // Each argument to a call takes on average one instruction to set up. |
| 102 | NumInsts += CS.arg_size(); |
Jakob Stoklund Olesen | 8b3ca84 | 2010-05-26 22:40:28 +0000 | [diff] [blame] | 103 | |
| 104 | // We don't want inline asm to count as a call - that would prevent loop |
| 105 | // unrolling. The argument setup cost is still real, though. |
| 106 | if (!isa<InlineAsm>(CS.getCalledValue())) |
| 107 | ++NumCalls; |
Jakob Stoklund Olesen | aa034fa | 2010-02-05 23:21:18 +0000 | [diff] [blame] | 108 | } |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 109 | } |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 110 | |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 111 | if (const AllocaInst *AI = dyn_cast<AllocaInst>(II)) { |
| 112 | if (!AI->isStaticAlloca()) |
| 113 | this->usesDynamicAlloca = true; |
| 114 | } |
| 115 | |
Duncan Sands | 1df9859 | 2010-02-16 11:11:14 +0000 | [diff] [blame] | 116 | if (isa<ExtractElementInst>(II) || II->getType()->isVectorTy()) |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 117 | ++NumVectorInsts; |
| 118 | |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 119 | if (const CastInst *CI = dyn_cast<CastInst>(II)) { |
Evan Cheng | 1a67dd2 | 2010-01-14 21:04:31 +0000 | [diff] [blame] | 120 | // Noop casts, including ptr <-> int, don't count. |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 121 | if (CI->isLosslessCast() || isa<IntToPtrInst>(CI) || |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 122 | isa<PtrToIntInst>(CI)) |
| 123 | continue; |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 124 | // trunc to a native type is free (assuming the target has compare and |
| 125 | // shift-right of the same width). |
| 126 | if (isa<TruncInst>(CI) && TD && |
| 127 | TD->isLegalInteger(TD->getTypeSizeInBits(CI->getType()))) |
| 128 | continue; |
Evan Cheng | 1a67dd2 | 2010-01-14 21:04:31 +0000 | [diff] [blame] | 129 | // Result of a cmp instruction is often extended (to be used by other |
| 130 | // cmp instructions, logical or return instructions). These are usually |
| 131 | // nop on most sane targets. |
| 132 | if (isa<CmpInst>(CI->getOperand(0))) |
| 133 | continue; |
Chris Lattner | b93a23a | 2009-11-01 03:07:53 +0000 | [diff] [blame] | 134 | } else if (const GetElementPtrInst *GEPI = dyn_cast<GetElementPtrInst>(II)){ |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 135 | // If a GEP has all constant indices, it will probably be folded with |
| 136 | // a load/store. |
| 137 | if (GEPI->hasAllConstantIndices()) |
| 138 | continue; |
| 139 | } |
| 140 | |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 141 | ++NumInsts; |
| 142 | } |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 143 | |
Chris Lattner | b93a23a | 2009-11-01 03:07:53 +0000 | [diff] [blame] | 144 | if (isa<ReturnInst>(BB->getTerminator())) |
| 145 | ++NumRets; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 146 | |
Chris Lattner | 6658870 | 2009-11-01 18:16:30 +0000 | [diff] [blame] | 147 | // We never want to inline functions that contain an indirectbr. This is |
Duncan Sands | b046964 | 2009-11-01 19:12:43 +0000 | [diff] [blame] | 148 | // incorrect because all the blockaddress's (in static global initializers |
| 149 | // for example) would be referring to the original function, and this indirect |
Chris Lattner | 6658870 | 2009-11-01 18:16:30 +0000 | [diff] [blame] | 150 | // jump would jump from the inlined copy of the function into the original |
| 151 | // function which is extremely undefined behavior. |
Eli Friedman | 531afb1 | 2011-10-20 04:05:33 +0000 | [diff] [blame] | 152 | // FIXME: This logic isn't really right; we can safely inline functions |
| 153 | // with indirectbr's as long as no other function or global references the |
| 154 | // blockaddress of a block within the current function. And as a QOI issue, |
Nick Lewycky | 144bef4 | 2011-12-21 20:21:55 +0000 | [diff] [blame] | 155 | // if someone is using a blockaddress without an indirectbr, and that |
Eli Friedman | 531afb1 | 2011-10-20 04:05:33 +0000 | [diff] [blame] | 156 | // reference somehow ends up in another function or global, we probably |
| 157 | // don't want to inline this function. |
Chris Lattner | b93a23a | 2009-11-01 03:07:53 +0000 | [diff] [blame] | 158 | if (isa<IndirectBrInst>(BB->getTerminator())) |
Kenneth Uildriks | 42c7d23 | 2010-06-09 15:11:37 +0000 | [diff] [blame] | 159 | containsIndirectBr = true; |
Devang Patel | cbd0560 | 2010-03-13 00:10:20 +0000 | [diff] [blame] | 160 | |
| 161 | // Remember NumInsts for this BB. |
Devang Patel | afc33fa | 2010-03-13 01:05:02 +0000 | [diff] [blame] | 162 | NumBBInsts[BB] = NumInsts - NumInstsBeforeThisBB; |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 163 | } |
| 164 | |
Chandler Carruth | 6f130bf | 2012-03-08 02:04:19 +0000 | [diff] [blame] | 165 | unsigned InlineCostAnalyzer::FunctionInfo::countCodeReductionForConstant( |
| 166 | const CodeMetrics &Metrics, Value *V) { |
Owen Anderson | 082bf2a | 2010-09-09 16:56:42 +0000 | [diff] [blame] | 167 | unsigned Reduction = 0; |
| 168 | for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E;++UI){ |
| 169 | User *U = *UI; |
| 170 | if (isa<BranchInst>(U) || isa<SwitchInst>(U)) { |
| 171 | // We will be able to eliminate all but one of the successors. |
| 172 | const TerminatorInst &TI = cast<TerminatorInst>(*U); |
| 173 | const unsigned NumSucc = TI.getNumSuccessors(); |
| 174 | unsigned Instrs = 0; |
| 175 | for (unsigned I = 0; I != NumSucc; ++I) |
Chandler Carruth | 6f130bf | 2012-03-08 02:04:19 +0000 | [diff] [blame] | 176 | Instrs += Metrics.NumBBInsts.lookup(TI.getSuccessor(I)); |
Owen Anderson | 082bf2a | 2010-09-09 16:56:42 +0000 | [diff] [blame] | 177 | // We don't know which blocks will be eliminated, so use the average size. |
| 178 | Reduction += InlineConstants::InstrCost*Instrs*(NumSucc-1)/NumSucc; |
Owen Anderson | 082bf2a | 2010-09-09 16:56:42 +0000 | [diff] [blame] | 179 | } else { |
| 180 | // Figure out if this instruction will be removed due to simple constant |
| 181 | // propagation. |
| 182 | Instruction &Inst = cast<Instruction>(*U); |
| 183 | |
| 184 | // We can't constant propagate instructions which have effects or |
| 185 | // read memory. |
| 186 | // |
| 187 | // FIXME: It would be nice to capture the fact that a load from a |
| 188 | // pointer-to-constant-global is actually a *really* good thing to zap. |
| 189 | // Unfortunately, we don't know the pointer that may get propagated here, |
| 190 | // so we can't make this decision. |
| 191 | if (Inst.mayReadFromMemory() || Inst.mayHaveSideEffects() || |
| 192 | isa<AllocaInst>(Inst)) |
| 193 | continue; |
| 194 | |
| 195 | bool AllOperandsConstant = true; |
| 196 | for (unsigned i = 0, e = Inst.getNumOperands(); i != e; ++i) |
| 197 | if (!isa<Constant>(Inst.getOperand(i)) && Inst.getOperand(i) != V) { |
| 198 | AllOperandsConstant = false; |
| 199 | break; |
| 200 | } |
| 201 | |
| 202 | if (AllOperandsConstant) { |
| 203 | // We will get to remove this instruction... |
| 204 | Reduction += InlineConstants::InstrCost; |
| 205 | |
| 206 | // And any other instructions that use it which become constants |
| 207 | // themselves. |
Chandler Carruth | 6f130bf | 2012-03-08 02:04:19 +0000 | [diff] [blame] | 208 | Reduction += countCodeReductionForConstant(Metrics, &Inst); |
Owen Anderson | 082bf2a | 2010-09-09 16:56:42 +0000 | [diff] [blame] | 209 | } |
| 210 | } |
| 211 | } |
| 212 | return Reduction; |
| 213 | } |
| 214 | |
Chandler Carruth | e8187e0 | 2012-03-09 02:49:36 +0000 | [diff] [blame^] | 215 | static unsigned countCodeReductionForAllocaICmp(const CodeMetrics &Metrics, |
| 216 | ICmpInst *ICI) { |
| 217 | unsigned Reduction = 0; |
| 218 | |
| 219 | // Bail if this is comparing against a non-constant; there is nothing we can |
| 220 | // do there. |
| 221 | if (!isa<Constant>(ICI->getOperand(1))) |
| 222 | return Reduction; |
| 223 | |
| 224 | // An icmp pred (alloca, C) becomes true if the predicate is true when |
| 225 | // equal and false otherwise. |
| 226 | bool Result = ICI->isTrueWhenEqual(); |
| 227 | |
| 228 | SmallVector<Instruction *, 4> Worklist; |
| 229 | Worklist.push_back(ICI); |
| 230 | do { |
| 231 | Instruction *U = Worklist.pop_back_val(); |
| 232 | Reduction += InlineConstants::InstrCost; |
| 233 | for (Value::use_iterator UI = U->use_begin(), UE = U->use_end(); |
| 234 | UI != UE; ++UI) { |
| 235 | Instruction *I = dyn_cast<Instruction>(*UI); |
| 236 | if (!I || I->mayHaveSideEffects()) continue; |
| 237 | if (I->getNumOperands() == 1) |
| 238 | Worklist.push_back(I); |
| 239 | if (BinaryOperator *BO = dyn_cast<BinaryOperator>(I)) { |
| 240 | // If BO produces the same value as U, then the other operand is |
| 241 | // irrelevant and we can put it into the Worklist to continue |
| 242 | // deleting dead instructions. If BO produces the same value as the |
| 243 | // other operand, we can delete BO but that's it. |
| 244 | if (Result == true) { |
| 245 | if (BO->getOpcode() == Instruction::Or) |
| 246 | Worklist.push_back(I); |
| 247 | if (BO->getOpcode() == Instruction::And) |
| 248 | Reduction += InlineConstants::InstrCost; |
| 249 | } else { |
| 250 | if (BO->getOpcode() == Instruction::Or || |
| 251 | BO->getOpcode() == Instruction::Xor) |
| 252 | Reduction += InlineConstants::InstrCost; |
| 253 | if (BO->getOpcode() == Instruction::And) |
| 254 | Worklist.push_back(I); |
| 255 | } |
| 256 | } |
| 257 | if (BranchInst *BI = dyn_cast<BranchInst>(I)) { |
| 258 | BasicBlock *BB = BI->getSuccessor(Result ? 0 : 1); |
| 259 | if (BB->getSinglePredecessor()) |
| 260 | Reduction |
| 261 | += InlineConstants::InstrCost * Metrics.NumBBInsts.lookup(BB); |
| 262 | } |
| 263 | } |
| 264 | } while (!Worklist.empty()); |
| 265 | |
| 266 | return Reduction; |
| 267 | } |
| 268 | |
| 269 | /// \brief Compute the reduction possible for a given instruction if we are able |
| 270 | /// to SROA an alloca. |
| 271 | /// |
| 272 | /// The reduction for this instruction is added to the SROAReduction output |
| 273 | /// parameter. Returns false if this instruction is expected to defeat SROA in |
| 274 | /// general. |
| 275 | bool countCodeReductionForSROAInst(Instruction *I, |
| 276 | SmallVectorImpl<Value *> &Worklist, |
| 277 | unsigned &SROAReduction) { |
| 278 | if (LoadInst *LI = dyn_cast<LoadInst>(I)) { |
| 279 | if (!LI->isSimple()) |
| 280 | return false; |
| 281 | SROAReduction += InlineConstants::InstrCost; |
| 282 | return true; |
| 283 | } |
| 284 | |
| 285 | if (StoreInst *SI = dyn_cast<StoreInst>(I)) { |
| 286 | if (!SI->isSimple()) |
| 287 | return false; |
| 288 | SROAReduction += InlineConstants::InstrCost; |
| 289 | return true; |
| 290 | } |
| 291 | |
| 292 | if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(I)) { |
| 293 | // If the GEP has variable indices, we won't be able to do much with it. |
| 294 | if (!GEP->hasAllConstantIndices()) |
| 295 | return false; |
| 296 | // A non-zero GEP will likely become a mask operation after SROA. |
| 297 | if (GEP->hasAllZeroIndices()) |
| 298 | SROAReduction += InlineConstants::InstrCost; |
| 299 | Worklist.push_back(GEP); |
| 300 | return true; |
| 301 | } |
| 302 | |
| 303 | if (BitCastInst *BCI = dyn_cast<BitCastInst>(I)) { |
| 304 | // Track pointer through bitcasts. |
| 305 | Worklist.push_back(BCI); |
| 306 | SROAReduction += InlineConstants::InstrCost; |
| 307 | return true; |
| 308 | } |
| 309 | |
| 310 | // We just look for non-constant operands to ICmp instructions as those will |
| 311 | // defeat SROA. The actual reduction for these happens even without SROA. |
| 312 | if (ICmpInst *ICI = dyn_cast<ICmpInst>(I)) |
| 313 | return isa<Constant>(ICI->getOperand(1)); |
| 314 | |
| 315 | if (SelectInst *SI = dyn_cast<SelectInst>(I)) { |
| 316 | // SROA can handle a select of alloca iff all uses of the alloca are |
| 317 | // loads, and dereferenceable. We assume it's dereferenceable since |
| 318 | // we're told the input is an alloca. |
| 319 | for (Value::use_iterator UI = SI->use_begin(), UE = SI->use_end(); |
| 320 | UI != UE; ++UI) { |
| 321 | LoadInst *LI = dyn_cast<LoadInst>(*UI); |
| 322 | if (LI == 0 || !LI->isSimple()) |
| 323 | return false; |
| 324 | } |
| 325 | // We don't know whether we'll be deleting the rest of the chain of |
| 326 | // instructions from the SelectInst on, because we don't know whether |
| 327 | // the other side of the select is also an alloca or not. |
| 328 | return true; |
| 329 | } |
| 330 | |
| 331 | if (IntrinsicInst *II = dyn_cast<IntrinsicInst>(I)) { |
| 332 | switch (II->getIntrinsicID()) { |
| 333 | default: |
| 334 | return false; |
| 335 | case Intrinsic::memset: |
| 336 | case Intrinsic::memcpy: |
| 337 | case Intrinsic::memmove: |
| 338 | case Intrinsic::lifetime_start: |
| 339 | case Intrinsic::lifetime_end: |
| 340 | // SROA can usually chew through these intrinsics. |
| 341 | SROAReduction += InlineConstants::InstrCost; |
| 342 | return true; |
| 343 | } |
| 344 | } |
| 345 | |
| 346 | // If there is some other strange instruction, we're not going to be |
| 347 | // able to do much if we inline this. |
| 348 | return false; |
| 349 | } |
| 350 | |
Chandler Carruth | 6f130bf | 2012-03-08 02:04:19 +0000 | [diff] [blame] | 351 | unsigned InlineCostAnalyzer::FunctionInfo::countCodeReductionForAlloca( |
| 352 | const CodeMetrics &Metrics, Value *V) { |
Owen Anderson | 082bf2a | 2010-09-09 16:56:42 +0000 | [diff] [blame] | 353 | if (!V->getType()->isPointerTy()) return 0; // Not a pointer |
| 354 | unsigned Reduction = 0; |
Chandler Carruth | e8187e0 | 2012-03-09 02:49:36 +0000 | [diff] [blame^] | 355 | unsigned SROAReduction = 0; |
| 356 | bool CanSROAAlloca = true; |
Nick Lewycky | 6977e79 | 2012-01-25 08:27:40 +0000 | [diff] [blame] | 357 | |
Nick Lewycky | 38b6d9d | 2012-01-20 08:35:20 +0000 | [diff] [blame] | 358 | SmallVector<Value *, 4> Worklist; |
| 359 | Worklist.push_back(V); |
| 360 | do { |
| 361 | Value *V = Worklist.pop_back_val(); |
| 362 | for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); |
| 363 | UI != E; ++UI){ |
| 364 | Instruction *I = cast<Instruction>(*UI); |
Chandler Carruth | e8187e0 | 2012-03-09 02:49:36 +0000 | [diff] [blame^] | 365 | |
| 366 | if (ICmpInst *ICI = dyn_cast<ICmpInst>(I)) |
| 367 | Reduction += countCodeReductionForAllocaICmp(Metrics, ICI); |
| 368 | |
| 369 | if (CanSROAAlloca) |
| 370 | CanSROAAlloca = countCodeReductionForSROAInst(I, Worklist, |
| 371 | SROAReduction); |
Owen Anderson | 082bf2a | 2010-09-09 16:56:42 +0000 | [diff] [blame] | 372 | } |
Nick Lewycky | 38b6d9d | 2012-01-20 08:35:20 +0000 | [diff] [blame] | 373 | } while (!Worklist.empty()); |
Owen Anderson | 082bf2a | 2010-09-09 16:56:42 +0000 | [diff] [blame] | 374 | |
Chandler Carruth | e8187e0 | 2012-03-09 02:49:36 +0000 | [diff] [blame^] | 375 | return Reduction + (CanSROAAlloca ? SROAReduction : 0); |
Owen Anderson | 082bf2a | 2010-09-09 16:56:42 +0000 | [diff] [blame] | 376 | } |
| 377 | |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 378 | /// analyzeFunction - Fill in the current structure with information gleaned |
| 379 | /// from the specified function. |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 380 | void CodeMetrics::analyzeFunction(Function *F, const TargetData *TD) { |
Bill Wendling | 728662f | 2011-10-17 18:22:52 +0000 | [diff] [blame] | 381 | // If this function contains a call that "returns twice" (e.g., setjmp or |
Joerg Sonnenberger | 3470693 | 2011-12-18 20:35:43 +0000 | [diff] [blame] | 382 | // _setjmp) and it isn't marked with "returns twice" itself, never inline it. |
| 383 | // This is a hack because we depend on the user marking their local variables |
| 384 | // as volatile if they are live across a setjmp call, and they probably |
| 385 | // won't do this in callers. |
| 386 | exposesReturnsTwice = F->callsFunctionThatReturnsTwice() && |
| 387 | !F->hasFnAttr(Attribute::ReturnsTwice); |
Rafael Espindola | ac53b0a | 2011-05-16 15:48:45 +0000 | [diff] [blame] | 388 | |
Dan Gohman | e7f0ed5 | 2009-10-13 19:58:07 +0000 | [diff] [blame] | 389 | // Look at the size of the callee. |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 390 | for (Function::const_iterator BB = F->begin(), E = F->end(); BB != E; ++BB) |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 391 | analyzeBasicBlock(&*BB, TD); |
Dan Gohman | e7f0ed5 | 2009-10-13 19:58:07 +0000 | [diff] [blame] | 392 | } |
| 393 | |
| 394 | /// analyzeFunction - Fill in the current structure with information gleaned |
| 395 | /// from the specified function. |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 396 | void InlineCostAnalyzer::FunctionInfo::analyzeFunction(Function *F, |
| 397 | const TargetData *TD) { |
| 398 | Metrics.analyzeFunction(F, TD); |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 399 | |
| 400 | // A function with exactly one return has it removed during the inlining |
| 401 | // process (see InlineFunction), so don't count it. |
Dan Gohman | e7f0ed5 | 2009-10-13 19:58:07 +0000 | [diff] [blame] | 402 | // FIXME: This knowledge should really be encoded outside of FunctionInfo. |
| 403 | if (Metrics.NumRets==1) |
| 404 | --Metrics.NumInsts; |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 405 | |
| 406 | // Check out all of the arguments to the function, figuring out how much |
| 407 | // code can be eliminated if one of the arguments is a constant. |
Jakob Stoklund Olesen | e3039b6 | 2010-01-26 21:31:24 +0000 | [diff] [blame] | 408 | ArgumentWeights.reserve(F->arg_size()); |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 409 | for (Function::arg_iterator I = F->arg_begin(), E = F->arg_end(); I != E; ++I) |
Chandler Carruth | 6f130bf | 2012-03-08 02:04:19 +0000 | [diff] [blame] | 410 | ArgumentWeights.push_back(ArgInfo(countCodeReductionForConstant(Metrics, I), |
| 411 | countCodeReductionForAlloca(Metrics, I))); |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 412 | } |
| 413 | |
Kenneth Uildriks | 42c7d23 | 2010-06-09 15:11:37 +0000 | [diff] [blame] | 414 | /// NeverInline - returns true if the function should never be inlined into |
| 415 | /// any caller |
Eric Christopher | 8e2da0c | 2011-02-01 01:16:32 +0000 | [diff] [blame] | 416 | bool InlineCostAnalyzer::FunctionInfo::NeverInline() { |
Joerg Sonnenberger | 3470693 | 2011-12-18 20:35:43 +0000 | [diff] [blame] | 417 | return (Metrics.exposesReturnsTwice || Metrics.isRecursive || |
Kenneth Uildriks | 42c7d23 | 2010-06-09 15:11:37 +0000 | [diff] [blame] | 418 | Metrics.containsIndirectBr); |
Kenneth Uildriks | 42c7d23 | 2010-06-09 15:11:37 +0000 | [diff] [blame] | 419 | } |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 420 | // getSpecializationBonus - The heuristic used to determine the per-call |
| 421 | // performance boost for using a specialization of Callee with argument |
| 422 | // specializedArgNo replaced by a constant. |
| 423 | int InlineCostAnalyzer::getSpecializationBonus(Function *Callee, |
| 424 | SmallVectorImpl<unsigned> &SpecializedArgNos) |
| 425 | { |
| 426 | if (Callee->mayBeOverridden()) |
| 427 | return 0; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 428 | |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 429 | int Bonus = 0; |
| 430 | // If this function uses the coldcc calling convention, prefer not to |
| 431 | // specialize it. |
| 432 | if (Callee->getCallingConv() == CallingConv::Cold) |
| 433 | Bonus -= InlineConstants::ColdccPenalty; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 434 | |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 435 | // Get information about the callee. |
| 436 | FunctionInfo *CalleeFI = &CachedFunctionInfo[Callee]; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 437 | |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 438 | // If we haven't calculated this information yet, do so now. |
| 439 | if (CalleeFI->Metrics.NumBlocks == 0) |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 440 | CalleeFI->analyzeFunction(Callee, TD); |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 441 | |
Eric Christopher | 8e2da0c | 2011-02-01 01:16:32 +0000 | [diff] [blame] | 442 | unsigned ArgNo = 0; |
| 443 | unsigned i = 0; |
| 444 | for (Function::arg_iterator I = Callee->arg_begin(), E = Callee->arg_end(); |
| 445 | I != E; ++I, ++ArgNo) |
| 446 | if (ArgNo == SpecializedArgNos[i]) { |
| 447 | ++i; |
| 448 | Bonus += CountBonusForConstant(I); |
| 449 | } |
Eric Christopher | 7d3a16f | 2011-01-26 01:09:59 +0000 | [diff] [blame] | 450 | |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 451 | // Calls usually take a long time, so they make the specialization gain |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 452 | // smaller. |
| 453 | Bonus -= CalleeFI->Metrics.NumCalls * InlineConstants::CallPenalty; |
| 454 | |
| 455 | return Bonus; |
| 456 | } |
| 457 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 458 | // ConstantFunctionBonus - Figure out how much of a bonus we can get for |
| 459 | // possibly devirtualizing a function. We'll subtract the size of the function |
| 460 | // we may wish to inline from the indirect call bonus providing a limit on |
| 461 | // growth. Leave an upper limit of 0 for the bonus - we don't want to penalize |
| 462 | // inlining because we decide we don't want to give a bonus for |
| 463 | // devirtualizing. |
| 464 | int InlineCostAnalyzer::ConstantFunctionBonus(CallSite CS, Constant *C) { |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 465 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 466 | // This could just be NULL. |
| 467 | if (!C) return 0; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 468 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 469 | Function *F = dyn_cast<Function>(C); |
| 470 | if (!F) return 0; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 471 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 472 | int Bonus = InlineConstants::IndirectCallBonus + getInlineSize(CS, F); |
| 473 | return (Bonus > 0) ? 0 : Bonus; |
| 474 | } |
| 475 | |
Eric Christopher | 8e2da0c | 2011-02-01 01:16:32 +0000 | [diff] [blame] | 476 | // CountBonusForConstant - Figure out an approximation for how much per-call |
| 477 | // performance boost we can expect if the specified value is constant. |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 478 | int InlineCostAnalyzer::CountBonusForConstant(Value *V, Constant *C) { |
Eric Christopher | 8e2da0c | 2011-02-01 01:16:32 +0000 | [diff] [blame] | 479 | unsigned Bonus = 0; |
Eric Christopher | 8e2da0c | 2011-02-01 01:16:32 +0000 | [diff] [blame] | 480 | for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E;++UI){ |
| 481 | User *U = *UI; |
| 482 | if (CallInst *CI = dyn_cast<CallInst>(U)) { |
| 483 | // Turning an indirect call into a direct call is a BIG win |
| 484 | if (CI->getCalledValue() == V) |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 485 | Bonus += ConstantFunctionBonus(CallSite(CI), C); |
| 486 | } else if (InvokeInst *II = dyn_cast<InvokeInst>(U)) { |
Eric Christopher | 8e2da0c | 2011-02-01 01:16:32 +0000 | [diff] [blame] | 487 | // Turning an indirect call into a direct call is a BIG win |
| 488 | if (II->getCalledValue() == V) |
Eric Christopher | a818d03 | 2011-02-05 02:48:47 +0000 | [diff] [blame] | 489 | Bonus += ConstantFunctionBonus(CallSite(II), C); |
Eric Christopher | 8e2da0c | 2011-02-01 01:16:32 +0000 | [diff] [blame] | 490 | } |
| 491 | // FIXME: Eliminating conditional branches and switches should |
| 492 | // also yield a per-call performance boost. |
| 493 | else { |
| 494 | // Figure out the bonuses that wll accrue due to simple constant |
| 495 | // propagation. |
| 496 | Instruction &Inst = cast<Instruction>(*U); |
| 497 | |
| 498 | // We can't constant propagate instructions which have effects or |
| 499 | // read memory. |
| 500 | // |
| 501 | // FIXME: It would be nice to capture the fact that a load from a |
| 502 | // pointer-to-constant-global is actually a *really* good thing to zap. |
| 503 | // Unfortunately, we don't know the pointer that may get propagated here, |
| 504 | // so we can't make this decision. |
| 505 | if (Inst.mayReadFromMemory() || Inst.mayHaveSideEffects() || |
| 506 | isa<AllocaInst>(Inst)) |
| 507 | continue; |
| 508 | |
| 509 | bool AllOperandsConstant = true; |
| 510 | for (unsigned i = 0, e = Inst.getNumOperands(); i != e; ++i) |
| 511 | if (!isa<Constant>(Inst.getOperand(i)) && Inst.getOperand(i) != V) { |
| 512 | AllOperandsConstant = false; |
| 513 | break; |
| 514 | } |
| 515 | |
| 516 | if (AllOperandsConstant) |
| 517 | Bonus += CountBonusForConstant(&Inst); |
| 518 | } |
| 519 | } |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 520 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 521 | return Bonus; |
| 522 | } |
| 523 | |
| 524 | int InlineCostAnalyzer::getInlineSize(CallSite CS, Function *Callee) { |
| 525 | // Get information about the callee. |
| 526 | FunctionInfo *CalleeFI = &CachedFunctionInfo[Callee]; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 527 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 528 | // If we haven't calculated this information yet, do so now. |
| 529 | if (CalleeFI->Metrics.NumBlocks == 0) |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 530 | CalleeFI->analyzeFunction(Callee, TD); |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 531 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 532 | // InlineCost - This value measures how good of an inline candidate this call |
| 533 | // site is to inline. A lower inline cost make is more likely for the call to |
| 534 | // be inlined. This value may go negative. |
| 535 | // |
| 536 | int InlineCost = 0; |
| 537 | |
| 538 | // Compute any size reductions we can expect due to arguments being passed into |
| 539 | // the function. |
| 540 | // |
| 541 | unsigned ArgNo = 0; |
| 542 | CallSite::arg_iterator I = CS.arg_begin(); |
| 543 | for (Function::arg_iterator FI = Callee->arg_begin(), FE = Callee->arg_end(); |
| 544 | FI != FE; ++I, ++FI, ++ArgNo) { |
| 545 | |
| 546 | // If an alloca is passed in, inlining this function is likely to allow |
| 547 | // significant future optimization possibilities (like scalar promotion, and |
| 548 | // scalarization), so encourage the inlining of the function. |
| 549 | // |
| 550 | if (isa<AllocaInst>(I)) |
| 551 | InlineCost -= CalleeFI->ArgumentWeights[ArgNo].AllocaWeight; |
| 552 | |
| 553 | // If this is a constant being passed into the function, use the argument |
| 554 | // weights calculated for the callee to determine how much will be folded |
| 555 | // away with this information. |
| 556 | else if (isa<Constant>(I)) |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 557 | InlineCost -= CalleeFI->ArgumentWeights[ArgNo].ConstantWeight; |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 558 | } |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 559 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 560 | // Each argument passed in has a cost at both the caller and the callee |
| 561 | // sides. Measurements show that each argument costs about the same as an |
| 562 | // instruction. |
| 563 | InlineCost -= (CS.arg_size() * InlineConstants::InstrCost); |
| 564 | |
| 565 | // Now that we have considered all of the factors that make the call site more |
| 566 | // likely to be inlined, look at factors that make us not want to inline it. |
| 567 | |
| 568 | // Calls usually take a long time, so they make the inlining gain smaller. |
| 569 | InlineCost += CalleeFI->Metrics.NumCalls * InlineConstants::CallPenalty; |
| 570 | |
| 571 | // Look at the size of the callee. Each instruction counts as 5. |
Nick Lewycky | 144bef4 | 2011-12-21 20:21:55 +0000 | [diff] [blame] | 572 | InlineCost += CalleeFI->Metrics.NumInsts * InlineConstants::InstrCost; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 573 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 574 | return InlineCost; |
| 575 | } |
| 576 | |
| 577 | int InlineCostAnalyzer::getInlineBonuses(CallSite CS, Function *Callee) { |
| 578 | // Get information about the callee. |
| 579 | FunctionInfo *CalleeFI = &CachedFunctionInfo[Callee]; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 580 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 581 | // If we haven't calculated this information yet, do so now. |
| 582 | if (CalleeFI->Metrics.NumBlocks == 0) |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 583 | CalleeFI->analyzeFunction(Callee, TD); |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 584 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 585 | bool isDirectCall = CS.getCalledFunction() == Callee; |
| 586 | Instruction *TheCall = CS.getInstruction(); |
| 587 | int Bonus = 0; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 588 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 589 | // If there is only one call of the function, and it has internal linkage, |
| 590 | // make it almost guaranteed to be inlined. |
| 591 | // |
| 592 | if (Callee->hasLocalLinkage() && Callee->hasOneUse() && isDirectCall) |
| 593 | Bonus += InlineConstants::LastCallToStaticBonus; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 594 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 595 | // If the instruction after the call, or if the normal destination of the |
| 596 | // invoke is an unreachable instruction, the function is noreturn. As such, |
| 597 | // there is little point in inlining this. |
| 598 | if (InvokeInst *II = dyn_cast<InvokeInst>(TheCall)) { |
| 599 | if (isa<UnreachableInst>(II->getNormalDest()->begin())) |
| 600 | Bonus += InlineConstants::NoreturnPenalty; |
| 601 | } else if (isa<UnreachableInst>(++BasicBlock::iterator(TheCall))) |
| 602 | Bonus += InlineConstants::NoreturnPenalty; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 603 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 604 | // If this function uses the coldcc calling convention, prefer not to inline |
| 605 | // it. |
| 606 | if (Callee->getCallingConv() == CallingConv::Cold) |
| 607 | Bonus += InlineConstants::ColdccPenalty; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 608 | |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 609 | // Add to the inline quality for properties that make the call valuable to |
| 610 | // inline. This includes factors that indicate that the result of inlining |
| 611 | // the function will be optimizable. Currently this just looks at arguments |
| 612 | // passed into the function. |
| 613 | // |
| 614 | CallSite::arg_iterator I = CS.arg_begin(); |
| 615 | for (Function::arg_iterator FI = Callee->arg_begin(), FE = Callee->arg_end(); |
| 616 | FI != FE; ++I, ++FI) |
| 617 | // Compute any constant bonus due to inlining we want to give here. |
| 618 | if (isa<Constant>(I)) |
| 619 | Bonus += CountBonusForConstant(FI, cast<Constant>(I)); |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 620 | |
Eric Christopher | 8e2da0c | 2011-02-01 01:16:32 +0000 | [diff] [blame] | 621 | return Bonus; |
| 622 | } |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 623 | |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 624 | // getInlineCost - The heuristic used to determine if we should inline the |
| 625 | // function call or not. |
| 626 | // |
| 627 | InlineCost InlineCostAnalyzer::getInlineCost(CallSite CS, |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 628 | SmallPtrSet<const Function*, 16> &NeverInline) { |
David Chisnall | 752e259 | 2010-05-01 15:47:41 +0000 | [diff] [blame] | 629 | return getInlineCost(CS, CS.getCalledFunction(), NeverInline); |
| 630 | } |
| 631 | |
| 632 | InlineCost InlineCostAnalyzer::getInlineCost(CallSite CS, |
| 633 | Function *Callee, |
| 634 | SmallPtrSet<const Function*, 16> &NeverInline) { |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 635 | Instruction *TheCall = CS.getInstruction(); |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 636 | Function *Caller = TheCall->getParent()->getParent(); |
| 637 | |
| 638 | // Don't inline functions which can be redefined at link-time to mean |
Eric Christopher | f27e608 | 2010-03-25 04:49:10 +0000 | [diff] [blame] | 639 | // something else. Don't inline functions marked noinline or call sites |
| 640 | // marked noinline. |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 641 | if (Callee->mayBeOverridden() || |
Eric Christopher | f27e608 | 2010-03-25 04:49:10 +0000 | [diff] [blame] | 642 | Callee->hasFnAttr(Attribute::NoInline) || NeverInline.count(Callee) || |
| 643 | CS.isNoInline()) |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 644 | return llvm::InlineCost::getNever(); |
| 645 | |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 646 | // Get information about the callee. |
| 647 | FunctionInfo *CalleeFI = &CachedFunctionInfo[Callee]; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 648 | |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 649 | // If we haven't calculated this information yet, do so now. |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 650 | if (CalleeFI->Metrics.NumBlocks == 0) |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 651 | CalleeFI->analyzeFunction(Callee, TD); |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 652 | |
| 653 | // If we should never inline this, return a huge cost. |
Kenneth Uildriks | 42c7d23 | 2010-06-09 15:11:37 +0000 | [diff] [blame] | 654 | if (CalleeFI->NeverInline()) |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 655 | return InlineCost::getNever(); |
| 656 | |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 657 | // FIXME: It would be nice to kill off CalleeFI->NeverInline. Then we |
Dan Gohman | e7f0ed5 | 2009-10-13 19:58:07 +0000 | [diff] [blame] | 658 | // could move this up and avoid computing the FunctionInfo for |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 659 | // things we are going to just return always inline for. This |
| 660 | // requires handling setjmp somewhere else, however. |
| 661 | if (!Callee->isDeclaration() && Callee->hasFnAttr(Attribute::AlwaysInline)) |
| 662 | return InlineCost::getAlways(); |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 663 | |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 664 | if (CalleeFI->Metrics.usesDynamicAlloca) { |
Chris Lattner | 7a2bdde | 2011-04-15 05:18:47 +0000 | [diff] [blame] | 665 | // Get information about the caller. |
Dan Gohman | e7f0ed5 | 2009-10-13 19:58:07 +0000 | [diff] [blame] | 666 | FunctionInfo &CallerFI = CachedFunctionInfo[Caller]; |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 667 | |
| 668 | // If we haven't calculated this information yet, do so now. |
Chris Lattner | f84755b | 2010-04-17 17:57:56 +0000 | [diff] [blame] | 669 | if (CallerFI.Metrics.NumBlocks == 0) { |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 670 | CallerFI.analyzeFunction(Caller, TD); |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 671 | |
Chris Lattner | f84755b | 2010-04-17 17:57:56 +0000 | [diff] [blame] | 672 | // Recompute the CalleeFI pointer, getting Caller could have invalidated |
| 673 | // it. |
| 674 | CalleeFI = &CachedFunctionInfo[Callee]; |
| 675 | } |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 676 | |
| 677 | // Don't inline a callee with dynamic alloca into a caller without them. |
| 678 | // Functions containing dynamic alloca's are inefficient in various ways; |
| 679 | // don't create more inefficiency. |
Dan Gohman | e7f0ed5 | 2009-10-13 19:58:07 +0000 | [diff] [blame] | 680 | if (!CallerFI.Metrics.usesDynamicAlloca) |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 681 | return InlineCost::getNever(); |
| 682 | } |
| 683 | |
Eric Christopher | 1bcb428 | 2011-01-25 01:34:31 +0000 | [diff] [blame] | 684 | // InlineCost - This value measures how good of an inline candidate this call |
| 685 | // site is to inline. A lower inline cost make is more likely for the call to |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 686 | // be inlined. This value may go negative due to the fact that bonuses |
| 687 | // are negative numbers. |
Eric Christopher | 1bcb428 | 2011-01-25 01:34:31 +0000 | [diff] [blame] | 688 | // |
Eric Christopher | 4e8af6d | 2011-02-05 00:49:15 +0000 | [diff] [blame] | 689 | int InlineCost = getInlineSize(CS, Callee) + getInlineBonuses(CS, Callee); |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 690 | return llvm::InlineCost::get(InlineCost); |
| 691 | } |
| 692 | |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 693 | // getSpecializationCost - The heuristic used to determine the code-size |
| 694 | // impact of creating a specialized version of Callee with argument |
| 695 | // SpecializedArgNo replaced by a constant. |
| 696 | InlineCost InlineCostAnalyzer::getSpecializationCost(Function *Callee, |
| 697 | SmallVectorImpl<unsigned> &SpecializedArgNos) |
| 698 | { |
| 699 | // Don't specialize functions which can be redefined at link-time to mean |
| 700 | // something else. |
| 701 | if (Callee->mayBeOverridden()) |
| 702 | return llvm::InlineCost::getNever(); |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 703 | |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 704 | // Get information about the callee. |
| 705 | FunctionInfo *CalleeFI = &CachedFunctionInfo[Callee]; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 706 | |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 707 | // If we haven't calculated this information yet, do so now. |
| 708 | if (CalleeFI->Metrics.NumBlocks == 0) |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 709 | CalleeFI->analyzeFunction(Callee, TD); |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 710 | |
| 711 | int Cost = 0; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 712 | |
Chris Lattner | 7a2bdde | 2011-04-15 05:18:47 +0000 | [diff] [blame] | 713 | // Look at the original size of the callee. Each instruction counts as 5. |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 714 | Cost += CalleeFI->Metrics.NumInsts * InlineConstants::InstrCost; |
| 715 | |
| 716 | // Offset that with the amount of code that can be constant-folded |
| 717 | // away with the given arguments replaced by constants. |
| 718 | for (SmallVectorImpl<unsigned>::iterator an = SpecializedArgNos.begin(), |
| 719 | ae = SpecializedArgNos.end(); an != ae; ++an) |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 720 | Cost -= CalleeFI->ArgumentWeights[*an].ConstantWeight; |
Kenneth Uildriks | 74fa732 | 2010-10-09 22:06:36 +0000 | [diff] [blame] | 721 | |
| 722 | return llvm::InlineCost::get(Cost); |
| 723 | } |
| 724 | |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 725 | // getInlineFudgeFactor - Return a > 1.0 factor if the inliner should use a |
| 726 | // higher threshold to determine if the function call should be inlined. |
| 727 | float InlineCostAnalyzer::getInlineFudgeFactor(CallSite CS) { |
| 728 | Function *Callee = CS.getCalledFunction(); |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 729 | |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 730 | // Get information about the callee. |
Dan Gohman | e7f0ed5 | 2009-10-13 19:58:07 +0000 | [diff] [blame] | 731 | FunctionInfo &CalleeFI = CachedFunctionInfo[Callee]; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 732 | |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 733 | // If we haven't calculated this information yet, do so now. |
Dan Gohman | e7f0ed5 | 2009-10-13 19:58:07 +0000 | [diff] [blame] | 734 | if (CalleeFI.Metrics.NumBlocks == 0) |
Andrew Trick | b2ab2fa | 2011-10-01 01:39:05 +0000 | [diff] [blame] | 735 | CalleeFI.analyzeFunction(Callee, TD); |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 736 | |
| 737 | float Factor = 1.0f; |
| 738 | // Single BB functions are often written to be inlined. |
Dan Gohman | e7f0ed5 | 2009-10-13 19:58:07 +0000 | [diff] [blame] | 739 | if (CalleeFI.Metrics.NumBlocks == 1) |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 740 | Factor += 0.5f; |
| 741 | |
| 742 | // Be more aggressive if the function contains a good chunk (if it mades up |
| 743 | // at least 10% of the instructions) of vector instructions. |
Dan Gohman | e7f0ed5 | 2009-10-13 19:58:07 +0000 | [diff] [blame] | 744 | if (CalleeFI.Metrics.NumVectorInsts > CalleeFI.Metrics.NumInsts/2) |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 745 | Factor += 2.0f; |
Dan Gohman | e7f0ed5 | 2009-10-13 19:58:07 +0000 | [diff] [blame] | 746 | else if (CalleeFI.Metrics.NumVectorInsts > CalleeFI.Metrics.NumInsts/10) |
Dan Gohman | e4aeec0 | 2009-10-13 18:30:07 +0000 | [diff] [blame] | 747 | Factor += 1.5f; |
| 748 | return Factor; |
| 749 | } |
Jakob Stoklund Olesen | f747747 | 2010-03-09 23:02:17 +0000 | [diff] [blame] | 750 | |
| 751 | /// growCachedCostInfo - update the cached cost info for Caller after Callee has |
| 752 | /// been inlined. |
| 753 | void |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 754 | InlineCostAnalyzer::growCachedCostInfo(Function *Caller, Function *Callee) { |
| 755 | CodeMetrics &CallerMetrics = CachedFunctionInfo[Caller].Metrics; |
Jakob Stoklund Olesen | f747747 | 2010-03-09 23:02:17 +0000 | [diff] [blame] | 756 | |
| 757 | // For small functions we prefer to recalculate the cost for better accuracy. |
Eli Friedman | b176399 | 2011-05-24 20:22:24 +0000 | [diff] [blame] | 758 | if (CallerMetrics.NumBlocks < 10 && CallerMetrics.NumInsts < 1000) { |
Jakob Stoklund Olesen | f747747 | 2010-03-09 23:02:17 +0000 | [diff] [blame] | 759 | resetCachedCostInfo(Caller); |
| 760 | return; |
| 761 | } |
| 762 | |
| 763 | // For large functions, we can save a lot of computation time by skipping |
| 764 | // recalculations. |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 765 | if (CallerMetrics.NumCalls > 0) |
| 766 | --CallerMetrics.NumCalls; |
Jakob Stoklund Olesen | f747747 | 2010-03-09 23:02:17 +0000 | [diff] [blame] | 767 | |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 768 | if (Callee == 0) return; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 769 | |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 770 | CodeMetrics &CalleeMetrics = CachedFunctionInfo[Callee].Metrics; |
Jakob Stoklund Olesen | f747747 | 2010-03-09 23:02:17 +0000 | [diff] [blame] | 771 | |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 772 | // If we don't have metrics for the callee, don't recalculate them just to |
| 773 | // update an approximation in the caller. Instead, just recalculate the |
| 774 | // caller info from scratch. |
| 775 | if (CalleeMetrics.NumBlocks == 0) { |
| 776 | resetCachedCostInfo(Caller); |
| 777 | return; |
Jakob Stoklund Olesen | f747747 | 2010-03-09 23:02:17 +0000 | [diff] [blame] | 778 | } |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 779 | |
Chris Lattner | f84755b | 2010-04-17 17:57:56 +0000 | [diff] [blame] | 780 | // Since CalleeMetrics were already calculated, we know that the CallerMetrics |
Kenneth Uildriks | 42c7d23 | 2010-06-09 15:11:37 +0000 | [diff] [blame] | 781 | // reference isn't invalidated: both were in the DenseMap. |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 782 | CallerMetrics.usesDynamicAlloca |= CalleeMetrics.usesDynamicAlloca; |
| 783 | |
Kenneth Uildriks | 42c7d23 | 2010-06-09 15:11:37 +0000 | [diff] [blame] | 784 | // FIXME: If any of these three are true for the callee, the callee was |
| 785 | // not inlined into the caller, so I think they're redundant here. |
Joerg Sonnenberger | 3470693 | 2011-12-18 20:35:43 +0000 | [diff] [blame] | 786 | CallerMetrics.exposesReturnsTwice |= CalleeMetrics.exposesReturnsTwice; |
Kenneth Uildriks | 42c7d23 | 2010-06-09 15:11:37 +0000 | [diff] [blame] | 787 | CallerMetrics.isRecursive |= CalleeMetrics.isRecursive; |
| 788 | CallerMetrics.containsIndirectBr |= CalleeMetrics.containsIndirectBr; |
| 789 | |
Chris Lattner | 44b04a5 | 2010-04-17 17:55:00 +0000 | [diff] [blame] | 790 | CallerMetrics.NumInsts += CalleeMetrics.NumInsts; |
| 791 | CallerMetrics.NumBlocks += CalleeMetrics.NumBlocks; |
| 792 | CallerMetrics.NumCalls += CalleeMetrics.NumCalls; |
| 793 | CallerMetrics.NumVectorInsts += CalleeMetrics.NumVectorInsts; |
| 794 | CallerMetrics.NumRets += CalleeMetrics.NumRets; |
| 795 | |
| 796 | // analyzeBasicBlock counts each function argument as an inst. |
| 797 | if (CallerMetrics.NumInsts >= Callee->arg_size()) |
| 798 | CallerMetrics.NumInsts -= Callee->arg_size(); |
| 799 | else |
| 800 | CallerMetrics.NumInsts = 0; |
Andrew Trick | 5c65541 | 2011-10-01 01:27:56 +0000 | [diff] [blame] | 801 | |
Nick Lewycky | 9a1581b | 2010-05-12 21:48:15 +0000 | [diff] [blame] | 802 | // We are not updating the argument weights. We have already determined that |
Jakob Stoklund Olesen | f747747 | 2010-03-09 23:02:17 +0000 | [diff] [blame] | 803 | // Caller is a fairly large function, so we accept the loss of precision. |
| 804 | } |
Nick Lewycky | 9a1581b | 2010-05-12 21:48:15 +0000 | [diff] [blame] | 805 | |
| 806 | /// clear - empty the cache of inline costs |
| 807 | void InlineCostAnalyzer::clear() { |
| 808 | CachedFunctionInfo.clear(); |
| 809 | } |