diff options
author | Tanya Lattner <tonic@nondot.org> | 2007-06-06 21:59:26 +0000 |
---|---|---|
committer | Tanya Lattner <tonic@nondot.org> | 2007-06-06 21:59:26 +0000 |
commit | cb90f1d881fd58547ae52edadd3ab57d6ab25b90 (patch) | |
tree | 9d76882a9c835abbd597b1b638e5a6509c70e4d0 /llvm/lib/Transforms | |
parent | c7c8b5a3d8e53f1720ee20f1e276dfe5382403a3 (diff) | |
download | bcm5719-llvm-cb90f1d881fd58547ae52edadd3ab57d6ab25b90.tar.gz bcm5719-llvm-cb90f1d881fd58547ae52edadd3ab57d6ab25b90.zip |
Instruct the inliner to obey the noinline attribute. Add test case.
llvm-svn: 37481
Diffstat (limited to 'llvm/lib/Transforms')
-rw-r--r-- | llvm/lib/Transforms/IPO/InlineSimple.cpp | 44 |
1 files changed, 43 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/IPO/InlineSimple.cpp b/llvm/lib/Transforms/IPO/InlineSimple.cpp index 14c19c9d685..3a3ad1999c3 100644 --- a/llvm/lib/Transforms/IPO/InlineSimple.cpp +++ b/llvm/lib/Transforms/IPO/InlineSimple.cpp @@ -15,11 +15,14 @@ #include "llvm/CallingConv.h" #include "llvm/Instructions.h" #include "llvm/IntrinsicInst.h" -#include "llvm/Function.h" +#include "llvm/Module.h" #include "llvm/Type.h" +#include "llvm/Analysis/CallGraph.h" #include "llvm/Support/CallSite.h" #include "llvm/Support/Compiler.h" #include "llvm/Transforms/IPO.h" +#include <set> + using namespace llvm; namespace { @@ -53,10 +56,12 @@ namespace { class VISIBILITY_HIDDEN SimpleInliner : public Inliner { std::map<const Function*, FunctionInfo> CachedFunctionInfo; + std::set<const Function*> NeverInline; // Functions that are never inlined public: SimpleInliner() : Inliner(&ID) {} static char ID; // Pass identification, replacement for typeid int getInlineCost(CallSite CS); + virtual bool doInitialization(CallGraph &CG); }; char SimpleInliner::ID = 0; RegisterPass<SimpleInliner> X("inline", "Function Integration/Inlining"); @@ -191,6 +196,9 @@ int SimpleInliner::getInlineCost(CallSite CS) { // Don't inline a directly recursive call. if (Caller == Callee) return 2000000000; + // Don't inline functions marked noinline + if (NeverInline.count(Callee)) return 2000000000; + // InlineCost - This value measures how good of an inline candidate this call // site is to inline. A lower inline cost make is more likely for the call to // be inlined. This value may go negative. @@ -274,3 +282,37 @@ int SimpleInliner::getInlineCost(CallSite CS) { return InlineCost; } +// doInitialization - Initializes the vector of functions that have been +// annotated with the noinline attribute. +bool SimpleInliner::doInitialization(CallGraph &CG) { + + Module &M = CG.getModule(); + + // Get llvm.noinline + GlobalVariable *GV = M.getNamedGlobal("llvm.noinline"); + + if(GV == 0) + return false; + + const ConstantArray *InitList = dyn_cast<ConstantArray>(GV->getInitializer()); + + if(InitList == 0) + return false; + + // Iterate over each element and add to the NeverInline set + for (unsigned i = 0, e = InitList->getNumOperands(); i != e; ++i) { + + // Get Source + const Constant *Elt = InitList->getOperand(i); + + if (const ConstantExpr *CE = dyn_cast<ConstantExpr>(Elt)) + if (CE->getOpcode() == Instruction::BitCast) + Elt = CE->getOperand(0); + + // Insert into set of functions to never inline + if(const Function *f = dyn_cast<Function>(Elt)) + NeverInline.insert(f); + } + + return false; +} |