Nick Lewycky | 586a7c4 | 2009-02-16 04:26:53 +0000 | [diff] [blame] | 1 | //===- InlineCost.cpp - Cost analysis for inliner -------------------------===// |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | 4ee451d | 2007-12-29 20:36:04 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
| 10 | // This file implements inline cost analysis. |
| 11 | // |
| 12 | //===----------------------------------------------------------------------===// |
| 13 | |
| 14 | |
| 15 | #include "llvm/Transforms/Utils/InlineCost.h" |
Victor Hernandez | 83d6391 | 2009-09-18 22:35:49 +0000 | [diff] [blame] | 16 | #include "llvm/Analysis/MallocHelper.h" |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 17 | #include "llvm/Support/CallSite.h" |
| 18 | #include "llvm/CallingConv.h" |
| 19 | #include "llvm/IntrinsicInst.h" |
Chris Lattner | 29f42ae | 2009-08-27 04:43:05 +0000 | [diff] [blame] | 20 | #include "llvm/ADT/SmallPtrSet.h" |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 21 | using namespace llvm; |
| 22 | |
| 23 | // CountCodeReductionForConstant - Figure out an approximation for how many |
| 24 | // instructions will be constant folded if the specified value is constant. |
| 25 | // |
| 26 | unsigned InlineCostAnalyzer::FunctionInfo:: |
| 27 | CountCodeReductionForConstant(Value *V) { |
| 28 | unsigned Reduction = 0; |
| 29 | for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E; ++UI) |
| 30 | if (isa<BranchInst>(*UI)) |
| 31 | Reduction += 40; // Eliminating a conditional branch is a big win |
| 32 | else if (SwitchInst *SI = dyn_cast<SwitchInst>(*UI)) |
| 33 | // Eliminating a switch is a big win, proportional to the number of edges |
| 34 | // deleted. |
| 35 | Reduction += (SI->getNumSuccessors()-1) * 40; |
| 36 | else if (CallInst *CI = dyn_cast<CallInst>(*UI)) { |
| 37 | // Turning an indirect call into a direct call is a BIG win |
| 38 | Reduction += CI->getCalledValue() == V ? 500 : 0; |
| 39 | } else if (InvokeInst *II = dyn_cast<InvokeInst>(*UI)) { |
| 40 | // Turning an indirect call into a direct call is a BIG win |
| 41 | Reduction += II->getCalledValue() == V ? 500 : 0; |
| 42 | } else { |
| 43 | // Figure out if this instruction will be removed due to simple constant |
| 44 | // propagation. |
| 45 | Instruction &Inst = cast<Instruction>(**UI); |
Eli Friedman | 4612e59 | 2009-07-18 05:26:06 +0000 | [diff] [blame] | 46 | |
| 47 | // We can't constant propagate instructions which have effects or |
| 48 | // read memory. |
Chris Lattner | 93f2491 | 2009-07-18 18:49:04 +0000 | [diff] [blame] | 49 | // |
| 50 | // FIXME: It would be nice to capture the fact that a load from a |
| 51 | // pointer-to-constant-global is actually a *really* good thing to zap. |
| 52 | // Unfortunately, we don't know the pointer that may get propagated here, |
| 53 | // so we can't make this decision. |
Eli Friedman | 4612e59 | 2009-07-18 05:26:06 +0000 | [diff] [blame] | 54 | if (Inst.mayReadFromMemory() || Inst.mayHaveSideEffects() || |
Victor Hernandez | 83d6391 | 2009-09-18 22:35:49 +0000 | [diff] [blame] | 55 | isa<AllocationInst>(Inst) || isMalloc(&Inst)) |
Eli Friedman | 4612e59 | 2009-07-18 05:26:06 +0000 | [diff] [blame] | 56 | continue; |
| 57 | |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 58 | bool AllOperandsConstant = true; |
| 59 | for (unsigned i = 0, e = Inst.getNumOperands(); i != e; ++i) |
| 60 | if (!isa<Constant>(Inst.getOperand(i)) && Inst.getOperand(i) != V) { |
| 61 | AllOperandsConstant = false; |
| 62 | break; |
| 63 | } |
| 64 | |
| 65 | if (AllOperandsConstant) { |
| 66 | // We will get to remove this instruction... |
| 67 | Reduction += 7; |
| 68 | |
| 69 | // And any other instructions that use it which become constants |
| 70 | // themselves. |
| 71 | Reduction += CountCodeReductionForConstant(&Inst); |
| 72 | } |
| 73 | } |
| 74 | |
| 75 | return Reduction; |
| 76 | } |
| 77 | |
| 78 | // CountCodeReductionForAlloca - Figure out an approximation of how much smaller |
| 79 | // the function will be if it is inlined into a context where an argument |
| 80 | // becomes an alloca. |
| 81 | // |
| 82 | unsigned InlineCostAnalyzer::FunctionInfo:: |
| 83 | CountCodeReductionForAlloca(Value *V) { |
| 84 | if (!isa<PointerType>(V->getType())) return 0; // Not a pointer |
| 85 | unsigned Reduction = 0; |
| 86 | for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI != E;++UI){ |
| 87 | Instruction *I = cast<Instruction>(*UI); |
| 88 | if (isa<LoadInst>(I) || isa<StoreInst>(I)) |
| 89 | Reduction += 10; |
| 90 | else if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(I)) { |
| 91 | // If the GEP has variable indices, we won't be able to do much with it. |
Chris Lattner | 41b1a48 | 2009-04-21 23:37:18 +0000 | [diff] [blame] | 92 | if (!GEP->hasAllConstantIndices()) |
| 93 | Reduction += CountCodeReductionForAlloca(GEP)+15; |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 94 | } else { |
| 95 | // If there is some other strange instruction, we're not going to be able |
| 96 | // to do much if we inline this. |
| 97 | return 0; |
| 98 | } |
| 99 | } |
| 100 | |
| 101 | return Reduction; |
| 102 | } |
| 103 | |
| 104 | /// analyzeFunction - Fill in the current structure with information gleaned |
| 105 | /// from the specified function. |
| 106 | void InlineCostAnalyzer::FunctionInfo::analyzeFunction(Function *F) { |
Dale Johannesen | d5405a6 | 2009-09-23 22:05:24 +0000 | [diff] [blame^] | 107 | unsigned NumInsts = 0, NumBlocks = 0, NumVectorInsts = 0, NumRets = 0; |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 108 | |
| 109 | // Look at the size of the callee. Each basic block counts as 20 units, and |
Devang Patel | 161660e | 2007-09-17 20:07:40 +0000 | [diff] [blame] | 110 | // each instruction counts as 5. |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 111 | for (Function::const_iterator BB = F->begin(), E = F->end(); BB != E; ++BB) { |
| 112 | for (BasicBlock::const_iterator II = BB->begin(), E = BB->end(); |
| 113 | II != E; ++II) { |
Evan Cheng | 8d84d5b | 2008-03-24 06:37:48 +0000 | [diff] [blame] | 114 | if (isa<PHINode>(II)) continue; // PHI nodes don't count. |
| 115 | |
Chris Lattner | 46868c0 | 2008-07-14 17:32:59 +0000 | [diff] [blame] | 116 | // Special handling for calls. |
| 117 | if (isa<CallInst>(II) || isa<InvokeInst>(II)) { |
| 118 | if (isa<DbgInfoIntrinsic>(II)) |
| 119 | continue; // Debug intrinsics don't count as size. |
| 120 | |
| 121 | CallSite CS = CallSite::get(const_cast<Instruction*>(&*II)); |
| 122 | |
| 123 | // If this function contains a call to setjmp or _setjmp, never inline |
| 124 | // it. This is a hack because we depend on the user marking their local |
| 125 | // variables as volatile if they are live across a setjmp call, and they |
| 126 | // probably won't do this in callers. |
| 127 | if (Function *F = CS.getCalledFunction()) |
| 128 | if (F->isDeclaration() && |
Daniel Dunbar | 03d7651 | 2009-07-25 23:55:21 +0000 | [diff] [blame] | 129 | (F->getName() == "setjmp" || F->getName() == "_setjmp")) { |
Chris Lattner | 46868c0 | 2008-07-14 17:32:59 +0000 | [diff] [blame] | 130 | NeverInline = true; |
| 131 | return; |
| 132 | } |
Dale Johannesen | 381e6f6 | 2009-01-24 21:49:34 +0000 | [diff] [blame] | 133 | |
Evan Cheng | 066fcf8 | 2008-07-17 01:31:49 +0000 | [diff] [blame] | 134 | // Calls often compile into many machine instructions. Bump up their |
Dale Johannesen | 381e6f6 | 2009-01-24 21:49:34 +0000 | [diff] [blame] | 135 | // cost to reflect this. |
| 136 | if (!isa<IntrinsicInst>(II)) |
| 137 | NumInsts += 5; |
Chris Lattner | 46868c0 | 2008-07-14 17:32:59 +0000 | [diff] [blame] | 138 | } |
| 139 | |
Dale Johannesen | 4362387 | 2009-01-08 21:45:23 +0000 | [diff] [blame] | 140 | if (const AllocaInst *AI = dyn_cast<AllocaInst>(II)) { |
Dale Johannesen | e345566 | 2009-01-09 01:30:11 +0000 | [diff] [blame] | 141 | if (!AI->isStaticAlloca()) |
Dale Johannesen | 4362387 | 2009-01-08 21:45:23 +0000 | [diff] [blame] | 142 | this->usesDynamicAlloca = true; |
| 143 | } |
| 144 | |
Chris Lattner | 4238453 | 2008-07-14 00:32:20 +0000 | [diff] [blame] | 145 | if (isa<ExtractElementInst>(II) || isa<VectorType>(II->getType())) |
Evan Cheng | 8d84d5b | 2008-03-24 06:37:48 +0000 | [diff] [blame] | 146 | ++NumVectorInsts; |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 147 | |
| 148 | // Noop casts, including ptr <-> int, don't count. |
| 149 | if (const CastInst *CI = dyn_cast<CastInst>(II)) { |
| 150 | if (CI->isLosslessCast() || isa<IntToPtrInst>(CI) || |
| 151 | isa<PtrToIntInst>(CI)) |
| 152 | continue; |
| 153 | } else if (const GetElementPtrInst *GEPI = |
Evan Cheng | 8d84d5b | 2008-03-24 06:37:48 +0000 | [diff] [blame] | 154 | dyn_cast<GetElementPtrInst>(II)) { |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 155 | // If a GEP has all constant indices, it will probably be folded with |
| 156 | // a load/store. |
Chris Lattner | 41b1a48 | 2009-04-21 23:37:18 +0000 | [diff] [blame] | 157 | if (GEPI->hasAllConstantIndices()) |
| 158 | continue; |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 159 | } |
Dale Johannesen | d5405a6 | 2009-09-23 22:05:24 +0000 | [diff] [blame^] | 160 | |
| 161 | if (isa<ReturnInst>(II)) |
| 162 | ++NumRets; |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 163 | |
| 164 | ++NumInsts; |
| 165 | } |
| 166 | |
| 167 | ++NumBlocks; |
| 168 | } |
| 169 | |
Dale Johannesen | d5405a6 | 2009-09-23 22:05:24 +0000 | [diff] [blame^] | 170 | // A function with exactly one return has it removed during the inlining |
| 171 | // process (see InlineFunction), so don't count it. |
| 172 | if (NumRets==1) |
| 173 | --NumInsts; |
| 174 | |
Evan Cheng | 8d84d5b | 2008-03-24 06:37:48 +0000 | [diff] [blame] | 175 | this->NumBlocks = NumBlocks; |
| 176 | this->NumInsts = NumInsts; |
| 177 | this->NumVectorInsts = NumVectorInsts; |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 178 | |
| 179 | // Check out all of the arguments to the function, figuring out how much |
| 180 | // code can be eliminated if one of the arguments is a constant. |
| 181 | for (Function::arg_iterator I = F->arg_begin(), E = F->arg_end(); I != E; ++I) |
| 182 | ArgumentWeights.push_back(ArgInfo(CountCodeReductionForConstant(I), |
| 183 | CountCodeReductionForAlloca(I))); |
| 184 | } |
| 185 | |
| 186 | |
| 187 | |
| 188 | // getInlineCost - The heuristic used to determine if we should inline the |
| 189 | // function call or not. |
| 190 | // |
Daniel Dunbar | c5e1ec4 | 2008-10-30 19:26:59 +0000 | [diff] [blame] | 191 | InlineCost InlineCostAnalyzer::getInlineCost(CallSite CS, |
Evan Cheng | 71d8374 | 2008-03-20 00:20:23 +0000 | [diff] [blame] | 192 | SmallPtrSet<const Function *, 16> &NeverInline) { |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 193 | Instruction *TheCall = CS.getInstruction(); |
| 194 | Function *Callee = CS.getCalledFunction(); |
Dale Johannesen | 4362387 | 2009-01-08 21:45:23 +0000 | [diff] [blame] | 195 | Function *Caller = TheCall->getParent()->getParent(); |
Duncan Sands | 5df3186 | 2008-09-29 11:25:42 +0000 | [diff] [blame] | 196 | |
Dale Johannesen | d5405a6 | 2009-09-23 22:05:24 +0000 | [diff] [blame^] | 197 | // Don't inline functions which can be redefined at link-time to mean |
| 198 | // something else. Don't inline functions marked noinline. |
| 199 | if (Callee->mayBeOverridden() || |
| 200 | Callee->hasFnAttr(Attribute::NoInline) || NeverInline.count(Callee)) |
Daniel Dunbar | c5e1ec4 | 2008-10-30 19:26:59 +0000 | [diff] [blame] | 201 | return llvm::InlineCost::getNever(); |
Duncan Sands | 5df3186 | 2008-09-29 11:25:42 +0000 | [diff] [blame] | 202 | |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 203 | // InlineCost - This value measures how good of an inline candidate this call |
| 204 | // site is to inline. A lower inline cost make is more likely for the call to |
| 205 | // be inlined. This value may go negative. |
| 206 | // |
| 207 | int InlineCost = 0; |
| 208 | |
| 209 | // If there is only one call of the function, and it has internal linkage, |
| 210 | // make it almost guaranteed to be inlined. |
| 211 | // |
Eli Friedman | 6dae757 | 2009-07-22 08:12:59 +0000 | [diff] [blame] | 212 | if (Callee->hasLocalLinkage() && Callee->hasOneUse()) |
Evan Cheng | 7932866 | 2008-04-24 18:42:47 +0000 | [diff] [blame] | 213 | InlineCost -= 15000; |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 214 | |
| 215 | // If this function uses the coldcc calling convention, prefer not to inline |
| 216 | // it. |
| 217 | if (Callee->getCallingConv() == CallingConv::Cold) |
| 218 | InlineCost += 2000; |
| 219 | |
| 220 | // If the instruction after the call, or if the normal destination of the |
| 221 | // invoke is an unreachable instruction, the function is noreturn. As such, |
| 222 | // there is little point in inlining this. |
| 223 | if (InvokeInst *II = dyn_cast<InvokeInst>(TheCall)) { |
| 224 | if (isa<UnreachableInst>(II->getNormalDest()->begin())) |
| 225 | InlineCost += 10000; |
| 226 | } else if (isa<UnreachableInst>(++BasicBlock::iterator(TheCall))) |
| 227 | InlineCost += 10000; |
| 228 | |
| 229 | // Get information about the callee... |
| 230 | FunctionInfo &CalleeFI = CachedFunctionInfo[Callee]; |
| 231 | |
| 232 | // If we haven't calculated this information yet, do so now. |
| 233 | if (CalleeFI.NumBlocks == 0) |
| 234 | CalleeFI.analyzeFunction(Callee); |
Dale Johannesen | 4362387 | 2009-01-08 21:45:23 +0000 | [diff] [blame] | 235 | |
Chris Lattner | 46868c0 | 2008-07-14 17:32:59 +0000 | [diff] [blame] | 236 | // If we should never inline this, return a huge cost. |
| 237 | if (CalleeFI.NeverInline) |
Daniel Dunbar | c5e1ec4 | 2008-10-30 19:26:59 +0000 | [diff] [blame] | 238 | return InlineCost::getNever(); |
Devang Patel | 6724339 | 2008-09-03 18:47:45 +0000 | [diff] [blame] | 239 | |
Evan Cheng | 8c7848f | 2009-03-10 07:57:50 +0000 | [diff] [blame] | 240 | // FIXME: It would be nice to kill off CalleeFI.NeverInline. Then we |
| 241 | // could move this up and avoid computing the FunctionInfo for |
| 242 | // things we are going to just return always inline for. This |
| 243 | // requires handling setjmp somewhere else, however. |
| 244 | if (!Callee->isDeclaration() && Callee->hasFnAttr(Attribute::AlwaysInline)) |
| 245 | return InlineCost::getAlways(); |
| 246 | |
Dale Johannesen | e345566 | 2009-01-09 01:30:11 +0000 | [diff] [blame] | 247 | if (CalleeFI.usesDynamicAlloca) { |
| 248 | // Get infomation about the caller... |
| 249 | FunctionInfo &CallerFI = CachedFunctionInfo[Caller]; |
Dale Johannesen | 4362387 | 2009-01-08 21:45:23 +0000 | [diff] [blame] | 250 | |
Dale Johannesen | e345566 | 2009-01-09 01:30:11 +0000 | [diff] [blame] | 251 | // If we haven't calculated this information yet, do so now. |
| 252 | if (CallerFI.NumBlocks == 0) |
| 253 | CallerFI.analyzeFunction(Caller); |
Dale Johannesen | 4362387 | 2009-01-08 21:45:23 +0000 | [diff] [blame] | 254 | |
Dale Johannesen | e345566 | 2009-01-09 01:30:11 +0000 | [diff] [blame] | 255 | // Don't inline a callee with dynamic alloca into a caller without them. |
| 256 | // Functions containing dynamic alloca's are inefficient in various ways; |
| 257 | // don't create more inefficiency. |
| 258 | if (!CallerFI.usesDynamicAlloca) |
| 259 | return InlineCost::getNever(); |
| 260 | } |
Dale Johannesen | 4362387 | 2009-01-08 21:45:23 +0000 | [diff] [blame] | 261 | |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 262 | // Add to the inline quality for properties that make the call valuable to |
| 263 | // inline. This includes factors that indicate that the result of inlining |
| 264 | // the function will be optimizable. Currently this just looks at arguments |
| 265 | // passed into the function. |
| 266 | // |
| 267 | unsigned ArgNo = 0; |
| 268 | for (CallSite::arg_iterator I = CS.arg_begin(), E = CS.arg_end(); |
| 269 | I != E; ++I, ++ArgNo) { |
| 270 | // Each argument passed in has a cost at both the caller and the callee |
| 271 | // sides. This favors functions that take many arguments over functions |
| 272 | // that take few arguments. |
| 273 | InlineCost -= 20; |
| 274 | |
| 275 | // If this is a function being passed in, it is very likely that we will be |
| 276 | // able to turn an indirect function call into a direct function call. |
| 277 | if (isa<Function>(I)) |
| 278 | InlineCost -= 100; |
| 279 | |
| 280 | // If an alloca is passed in, inlining this function is likely to allow |
| 281 | // significant future optimization possibilities (like scalar promotion, and |
| 282 | // scalarization), so encourage the inlining of the function. |
| 283 | // |
| 284 | else if (isa<AllocaInst>(I)) { |
| 285 | if (ArgNo < CalleeFI.ArgumentWeights.size()) |
| 286 | InlineCost -= CalleeFI.ArgumentWeights[ArgNo].AllocaWeight; |
| 287 | |
| 288 | // If this is a constant being passed into the function, use the argument |
| 289 | // weights calculated for the callee to determine how much will be folded |
| 290 | // away with this information. |
| 291 | } else if (isa<Constant>(I)) { |
| 292 | if (ArgNo < CalleeFI.ArgumentWeights.size()) |
| 293 | InlineCost -= CalleeFI.ArgumentWeights[ArgNo].ConstantWeight; |
| 294 | } |
| 295 | } |
| 296 | |
| 297 | // Now that we have considered all of the factors that make the call site more |
| 298 | // likely to be inlined, look at factors that make us not want to inline it. |
| 299 | |
Evan Cheng | 7c3becd | 2008-04-01 23:59:29 +0000 | [diff] [blame] | 300 | // Don't inline into something too big, which would make it bigger. |
Dale Johannesen | d5405a6 | 2009-09-23 22:05:24 +0000 | [diff] [blame^] | 301 | // "size" here is the number of basic blocks, not instructions. |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 302 | // |
Evan Cheng | 7932866 | 2008-04-24 18:42:47 +0000 | [diff] [blame] | 303 | InlineCost += Caller->size()/15; |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 304 | |
Evan Cheng | 7c3becd | 2008-04-01 23:59:29 +0000 | [diff] [blame] | 305 | // Look at the size of the callee. Each instruction counts as 5. |
| 306 | InlineCost += CalleeFI.NumInsts*5; |
Evan Cheng | 8d84d5b | 2008-03-24 06:37:48 +0000 | [diff] [blame] | 307 | |
Daniel Dunbar | c5e1ec4 | 2008-10-30 19:26:59 +0000 | [diff] [blame] | 308 | return llvm::InlineCost::get(InlineCost); |
Devang Patel | 6899b31 | 2007-07-25 18:00:25 +0000 | [diff] [blame] | 309 | } |
| 310 | |
Evan Cheng | 8d84d5b | 2008-03-24 06:37:48 +0000 | [diff] [blame] | 311 | // getInlineFudgeFactor - Return a > 1.0 factor if the inliner should use a |
| 312 | // higher threshold to determine if the function call should be inlined. |
| 313 | float InlineCostAnalyzer::getInlineFudgeFactor(CallSite CS) { |
| 314 | Function *Callee = CS.getCalledFunction(); |
| 315 | |
| 316 | // Get information about the callee... |
| 317 | FunctionInfo &CalleeFI = CachedFunctionInfo[Callee]; |
| 318 | |
| 319 | // If we haven't calculated this information yet, do so now. |
| 320 | if (CalleeFI.NumBlocks == 0) |
| 321 | CalleeFI.analyzeFunction(Callee); |
| 322 | |
Evan Cheng | 7c3becd | 2008-04-01 23:59:29 +0000 | [diff] [blame] | 323 | float Factor = 1.0f; |
| 324 | // Single BB functions are often written to be inlined. |
| 325 | if (CalleeFI.NumBlocks == 1) |
| 326 | Factor += 0.5f; |
| 327 | |
Evan Cheng | 8d84d5b | 2008-03-24 06:37:48 +0000 | [diff] [blame] | 328 | // Be more aggressive if the function contains a good chunk (if it mades up |
| 329 | // at least 10% of the instructions) of vector instructions. |
Evan Cheng | 7c3becd | 2008-04-01 23:59:29 +0000 | [diff] [blame] | 330 | if (CalleeFI.NumVectorInsts > CalleeFI.NumInsts/2) |
| 331 | Factor += 2.0f; |
| 332 | else if (CalleeFI.NumVectorInsts > CalleeFI.NumInsts/10) |
| 333 | Factor += 1.5f; |
| 334 | return Factor; |
Evan Cheng | 8d84d5b | 2008-03-24 06:37:48 +0000 | [diff] [blame] | 335 | } |