From 7b166d9969670665d9f093f20ed3381057427256 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Sat, 9 Sep 2006 20:40:44 +0000 Subject: [PATCH] Make inlining costs more accurate. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@30231 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/IPO/InlineSimple.cpp | 29 ++++++++++++++++++++++++++--- 1 file changed, 26 insertions(+), 3 deletions(-) diff --git a/lib/Transforms/IPO/InlineSimple.cpp b/lib/Transforms/IPO/InlineSimple.cpp index 64366848357..c2aa4cb3809 100644 --- a/lib/Transforms/IPO/InlineSimple.cpp +++ b/lib/Transforms/IPO/InlineSimple.cpp @@ -138,9 +138,32 @@ void FunctionInfo::analyzeFunction(Function *F) { // each instruction counts as 10. for (Function::const_iterator BB = F->begin(), E = F->end(); BB != E; ++BB) { for (BasicBlock::const_iterator II = BB->begin(), E = BB->end(); - II != E; ++II) - if (!isa(II)) - ++NumInsts; + II != E; ++II) { + if (isa(II)) continue; // Debug intrinsics don't count. + + // Noop casts don't count. + if (const CastInst *CI = dyn_cast(II)) { + const Type *OpTy = CI->getOperand(0)->getType(); + if (CI->getType()->isLosslesslyConvertibleTo(OpTy)) + continue; + if ((isa(CI->getType()) && OpTy->isInteger()) || + (isa(OpTy) && CI->getType()->isInteger())) + continue; // ptr <-> int is *probably* noop cast. + } else if (const GetElementPtrInst *GEPI = + dyn_cast(II)) { + // If a GEP has all constant indices, it will probably be folded with + // a load/store. + bool AllConstant = true; + for (unsigned i = 1, e = GEPI->getNumOperands(); i != e; ++i) + if (!isa(GEPI->getOperand(i))) { + AllConstant = false; + break; + } + if (AllConstant) continue; + } + + ++NumInsts; + } ++NumBlocks; }