From 315596c39cf67bf8fab9378bd6b3f4b94a0aaac2 Mon Sep 17 00:00:00 2001 From: Eli Friedman Date: Wed, 22 Jul 2009 08:12:59 +0000 Subject: 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. llvm-svn: 76730 --- llvm/lib/Transforms/Utils/InlineCost.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'llvm/lib/Transforms/Utils') diff --git a/llvm/lib/Transforms/Utils/InlineCost.cpp b/llvm/lib/Transforms/Utils/InlineCost.cpp index 8925a752b19..9cd47027631 100644 --- a/llvm/lib/Transforms/Utils/InlineCost.cpp +++ b/llvm/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 -- cgit v1.2.3