From 381e6f6b7d2e8321af30363fe435d5ec8863c6d5 Mon Sep 17 00:00:00 2001 From: Dale Johannesen Date: Sat, 24 Jan 2009 21:49:34 +0000 Subject: [PATCH] Revert previous change; even this mild and clearly more accurate change loses more than it gains on benchmarks. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@62938 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Utils/InlineCost.cpp | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/lib/Transforms/Utils/InlineCost.cpp b/lib/Transforms/Utils/InlineCost.cpp index 37d2c0352ac..69cd9343f8c 100644 --- a/lib/Transforms/Utils/InlineCost.cpp +++ b/lib/Transforms/Utils/InlineCost.cpp @@ -119,14 +119,11 @@ void InlineCostAnalyzer::FunctionInfo::analyzeFunction(Function *F) { NeverInline = true; return; } - + // Calls often compile into many machine instructions. Bump up their - // cost to reflect this. Allow an extra instruction per parameter. - if (!isa(II)) { - for (CallSite::arg_iterator I = CS.arg_begin(), E = CS.arg_end(); - I != E; ++I) - NumInsts ++; - } + // cost to reflect this. + if (!isa(II)) + NumInsts += 5; } if (const AllocaInst *AI = dyn_cast(II)) {