diff options
| author | Piotr Padlewski <piotr.padlewski@gmail.com> | 2016-08-10 21:15:22 +0000 |
|---|---|---|
| committer | Piotr Padlewski <piotr.padlewski@gmail.com> | 2016-08-10 21:15:22 +0000 |
| commit | d89875ca39f98e6df5a331b8c43eaa982822f0c4 (patch) | |
| tree | a049afe266ab5f6c993be1e43af9937fb54696e9 /llvm/lib/Analysis/InlineCost.cpp | |
| parent | c8b371734445cf1c5c19077580c3529998364d3e (diff) | |
| download | llvm-d89875ca39f98e6df5a331b8c43eaa982822f0c4.zip llvm-d89875ca39f98e6df5a331b8c43eaa982822f0c4.tar.gz llvm-d89875ca39f98e6df5a331b8c43eaa982822f0c4.tar.bz2 | |
Changed sign of LastCallToStaticBouns
Summary:
I think it is much better this way.
When I firstly saw line:
Cost += InlineConstants::LastCallToStaticBonus;
I though that this is a bug, because everywhere where the cost is being reduced
it is usuing -=.
Reviewers: eraman, tejohnson, mehdi_amini
Subscribers: llvm-commits, mehdi_amini
Differential Revision: https://reviews.llvm.org/D23222
llvm-svn: 278290
Diffstat (limited to 'llvm/lib/Analysis/InlineCost.cpp')
| -rw-r--r-- | llvm/lib/Analysis/InlineCost.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Analysis/InlineCost.cpp b/llvm/lib/Analysis/InlineCost.cpp index dd9174c..c0ac6c8 100644 --- a/llvm/lib/Analysis/InlineCost.cpp +++ b/llvm/lib/Analysis/InlineCost.cpp @@ -1257,7 +1257,7 @@ bool CallAnalyzer::analyzeCall(CallSite CS) { bool OnlyOneCallAndLocalLinkage = F.hasLocalLinkage() && F.hasOneUse() && &F == CS.getCalledFunction(); if (OnlyOneCallAndLocalLinkage) - Cost += InlineConstants::LastCallToStaticBonus; + Cost -= InlineConstants::LastCallToStaticBonus; // If this function uses the coldcc calling convention, prefer not to inline // it. |