From 910c120a7c439ee3c6e9cd249ff0a05f0fdb6c98 Mon Sep 17 00:00:00 2001 From: Devang Patel Date: Wed, 3 Sep 2008 18:10:21 +0000 Subject: [PATCH] Handle "noinline" note inside the simple inliner. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@55708 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/IPO/InlineSimple.cpp | 5 +++++ lib/Transforms/IPO/Inliner.cpp | 6 ++---- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/lib/Transforms/IPO/InlineSimple.cpp b/lib/Transforms/IPO/InlineSimple.cpp index 63a4798cce1..4171e22302d 100644 --- a/lib/Transforms/IPO/InlineSimple.cpp +++ b/lib/Transforms/IPO/InlineSimple.cpp @@ -63,6 +63,11 @@ bool SimpleInliner::doInitialization(CallGraph &CG) { Module &M = CG.getModule(); + for (Module::iterator I = M.begin(), E = M.end(); + I != E; ++I) + if (!I->isDeclaration() && I->getNotes() == FN_NOTE_NoInline) + NeverInline.insert(I); + // Get llvm.noinline GlobalVariable *GV = M.getNamedGlobal("llvm.noinline"); diff --git a/lib/Transforms/IPO/Inliner.cpp b/lib/Transforms/IPO/Inliner.cpp index 845ffd864e3..3ea16208a52 100644 --- a/lib/Transforms/IPO/Inliner.cpp +++ b/lib/Transforms/IPO/Inliner.cpp @@ -144,10 +144,8 @@ bool Inliner::runOnSCC(const std::vector &SCC) { bool AlwaysInline = false; if (Fn && (Fn->getNotes() & FN_NOTE_AlwaysInline)) AlwaysInline = true; - if (Fn && (Fn->getNotes() & FN_NOTE_NoInline)) - DOUT << "NOT Inlining: inline=never is set" << *CS.getInstruction(); - else if (!AlwaysInline - && InlineCost >= (int)(InlineThreshold * FudgeFactor)) { + if (!AlwaysInline + && InlineCost >= (int)(InlineThreshold * FudgeFactor)) { DOUT << " NOT Inlining: cost=" << InlineCost << ", Call: " << *CS.getInstruction(); } else { -- 2.34.1