From 2e03094d45e4ffa3aba3bcbef42f612c5b72bf2f 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. llvm-svn: 269251 --- llvm/lib/Transforms/IPO/FunctionImport.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/lib/Transforms/IPO/FunctionImport.cpp b/llvm/lib/Transforms/IPO/FunctionImport.cpp index 498ce5e3fb4d..a089f3c16e3e 100644 --- a/llvm/lib/Transforms/IPO/FunctionImport.cpp +++ b/llvm/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;