From afc33fa6d5f16089e7747ad761ae2d498ce43e90 Mon Sep 17 00:00:00 2001 From: Devang Patel Date: Sat, 13 Mar 2010 01:05:02 +0000 Subject: [PATCH] Do not ignore arg_size() impact while counting bb instructions. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@98408 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Analysis/InlineCost.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/Analysis/InlineCost.cpp b/lib/Analysis/InlineCost.cpp index f14dc5d3478..cf2ce0e06a7 100644 --- a/lib/Analysis/InlineCost.cpp +++ b/lib/Analysis/InlineCost.cpp @@ -142,7 +142,7 @@ static bool callIsSmall(const Function *F) { /// from the specified block. void CodeMetrics::analyzeBasicBlock(const BasicBlock *BB) { ++NumBlocks; - unsigned NumInstsInThisBB = 0; + unsigned NumInstsBeforeThisBB = NumInsts; for (BasicBlock::const_iterator II = BB->begin(), E = BB->end(); II != E; ++II) { if (isa(II)) continue; // PHI nodes don't count. @@ -196,7 +196,6 @@ void CodeMetrics::analyzeBasicBlock(const BasicBlock *BB) { } ++NumInsts; - ++NumInstsInThisBB; } if (isa(BB->getTerminator())) @@ -211,7 +210,7 @@ void CodeMetrics::analyzeBasicBlock(const BasicBlock *BB) { NeverInline = true; // Remember NumInsts for this BB. - NumBBInsts[BB] = NumInstsInThisBB; + NumBBInsts[BB] = NumInsts - NumInstsBeforeThisBB; } /// analyzeFunction - Fill in the current structure with information gleaned -- 2.34.1