RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-11-24 19:52:54 +01:00
Code Issues Projects Releases Wiki Activity

A minor improvment in accuracy to inline cost

Browse Source 
computation, and some cosmetics.

llvm-svn: 82660
This commit is contained in:
Dale Johannesen 2009-09-23 22:05:24 +00:00
parent 2805d91d12
commit 1b76c2ca48
1 changed files with 14 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
lib/Transforms/Utils/InlineCost.cpp
View File

@ -104,7 +104,7 @@ unsigned InlineCostAnalyzer::FunctionInfo::
/// analyzeFunction - Fill in the current structure with information gleaned
/// from the specified function.
void InlineCostAnalyzer::FunctionInfo::analyzeFunction(Function *F) {
unsigned NumInsts = 0, NumBlocks = 0, NumVectorInsts = 0;
unsigned NumInsts = 0, NumBlocks = 0, NumVectorInsts = 0, NumRets = 0;
// Look at the size of the callee. Each basic block counts as 20 units, and
// each instruction counts as 5.
@ -157,6 +157,9 @@ void InlineCostAnalyzer::FunctionInfo::analyzeFunction(Function *F) {
if (GEPI->hasAllConstantIndices())
continue;
}
if (isa<ReturnInst>(II))
++NumRets;
++NumInsts;
}
@ -164,6 +167,11 @@ void InlineCostAnalyzer::FunctionInfo::analyzeFunction(Function *F) {
++NumBlocks;
}
// A function with exactly one return has it removed during the inlining
// process (see InlineFunction), so don't count it.
if (NumRets==1)
--NumInsts;
this->NumBlocks = NumBlocks;
this->NumInsts = NumInsts;
this->NumVectorInsts = NumVectorInsts;
@ -186,11 +194,10 @@ InlineCost InlineCostAnalyzer::getInlineCost(CallSite CS,
Function *Callee = CS.getCalledFunction();
Function *Caller = TheCall->getParent()->getParent();
// Don't inline functions which can be redefined at link-time to mean
// something else.
if (Callee->mayBeOverridden() ||
// Don't inline functions marked noinline.
Callee->hasFnAttr(Attribute::NoInline) || NeverInline.count(Callee))
// Don't inline functions which can be redefined at link-time to mean
// something else. Don't inline functions marked noinline.
if (Callee->mayBeOverridden() ||
Callee->hasFnAttr(Attribute::NoInline) || NeverInline.count(Callee))
return llvm::InlineCost::getNever();
// InlineCost - This value measures how good of an inline candidate this call
@ -291,6 +298,7 @@ InlineCost InlineCostAnalyzer::getInlineCost(CallSite CS,
// likely to be inlined, look at factors that make us not want to inline it.
// Don't inline into something too big, which would make it bigger.
// "size" here is the number of basic blocks, not instructions.
//
InlineCost += Caller->size()/15;