diff options
| author | Eli Friedman <eli.friedman@gmail.com> | 2009-07-22 08:12:59 +0000 |
|---|---|---|
| committer | Eli Friedman <eli.friedman@gmail.com> | 2009-07-22 08:12:59 +0000 |
| commit | 6dae757a071fc033c1989b2e4eaca9718dc4302c (patch) | |
| tree | 3bde863c5e9ea2c4afe6fd5606b9cc831d5692a9 | |
| parent | 756da12ae4030259d98fefbe26f840d49f71898e (diff) | |
| download | external_llvm-6dae757a071fc033c1989b2e4eaca9718dc4302c.zip external_llvm-6dae757a071fc033c1989b2e4eaca9718dc4302c.tar.gz external_llvm-6dae757a071fc033c1989b2e4eaca9718dc4302c.tar.bz2 | |
Don't give a massive inlining cost bonus to available_externally
functions with a single use; eliminating the single use may eliminate
the function from the current module, but usually doesn't eliminate
it from the final program.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@76730 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/Transforms/Utils/InlineCost.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/Transforms/Utils/InlineCost.cpp b/lib/Transforms/Utils/InlineCost.cpp index 8925a75..9cd4702 100644 --- a/lib/Transforms/Utils/InlineCost.cpp +++ b/lib/Transforms/Utils/InlineCost.cpp @@ -201,8 +201,7 @@ InlineCost InlineCostAnalyzer::getInlineCost(CallSite CS, // If there is only one call of the function, and it has internal linkage, // make it almost guaranteed to be inlined. // - if ((Callee->hasLocalLinkage() || Callee->hasAvailableExternallyLinkage()) && - Callee->hasOneUse()) + if (Callee->hasLocalLinkage() && Callee->hasOneUse()) InlineCost -= 15000; // If this function uses the coldcc calling convention, prefer not to inline |