diff options
| author | Chris Lattner <sabre@nondot.org> | 2007-06-25 21:50:09 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2007-06-25 21:50:09 +0000 |
| commit | ba6801e6e72a9f4de1e116ea81b370456eb7ecd3 (patch) | |
| tree | ed347a2e7abf7bad61436d01b12dfddc1ee9aade | |
| parent | 4939debe2af6cd57e4be993bc70a3a5d2a678bbd (diff) | |
fix Transforms/Inline/2007-06-25-WeakInline.ll by not inlining functions
with weak linkage.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@37723 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/Transforms/IPO/InlineSimple.cpp | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/lib/Transforms/IPO/InlineSimple.cpp b/lib/Transforms/IPO/InlineSimple.cpp index 83cfe901a0..2157dcd2fe 100644 --- a/lib/Transforms/IPO/InlineSimple.cpp +++ b/lib/Transforms/IPO/InlineSimple.cpp @@ -195,10 +195,14 @@ int SimpleInliner::getInlineCost(CallSite CS) { const Function *Caller = TheCall->getParent()->getParent(); // Don't inline a directly recursive call. - if (Caller == Callee) return 2000000000; - - // Don't inline functions marked noinline - if (NeverInline.count(Callee)) return 2000000000; + if (Caller == Callee || + // Don't inline functions which can be redefined at link-time to mean + // something else. link-once linkage is ok though. + Callee->hasWeakLinkage() || + + // Don't inline functions marked noinline. + 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 |