From a8330647b24d8a79212f0378dd6b5f99e83e62e9 Mon Sep 17 00:00:00 2001 From: Teresa Johnson Date: Wed, 11 May 2016 22:56:19 +0000 Subject: [PATCH] [ThinLTO] Don't re-analyze callee at same threshold unnecessarily This should just be a compile-time change. Correct the check for whether we have already analyzed the callee when making summary based decisions. There is no need to reprocess one at the same threshold as when it was last processed. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@269251 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/IPO/FunctionImport.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/Transforms/IPO/FunctionImport.cpp b/lib/Transforms/IPO/FunctionImport.cpp index 498ce5e3fb4..a089f3c16e3 100644 --- a/lib/Transforms/IPO/FunctionImport.cpp +++ b/lib/Transforms/IPO/FunctionImport.cpp @@ -288,7 +288,7 @@ static void computeImportForFunction( /// Since the traversal of the call graph is DFS, we can revisit a function /// a second time with a higher threshold. In this case, it is added back to /// the worklist with the new threshold. - if (ProcessedThreshold && ProcessedThreshold > Threshold) { + if (ProcessedThreshold && ProcessedThreshold >= Threshold) { DEBUG(dbgs() << "ignored! Target was already seen with Threshold " << ProcessedThreshold << "\n"); continue; -- 2.11.0