summaryrefslogtreecommitdiffstats
path: root/llvm/unittests/Analysis/CGSCCPassManagerTest.cpp
diff options
context:
space:
mode:
authorChandler Carruth <chandlerc@gmail.com>2017-07-12 09:08:11 +0000
committerChandler Carruth <chandlerc@gmail.com>2017-07-12 09:08:11 +0000
commit051bdb0b22a3eba1d8eb97f071195a13559a965a (patch)
treedbef8d7f24de3d5eae48f5dba53f03942cd80610 /llvm/unittests/Analysis/CGSCCPassManagerTest.cpp
parent21014df5e091f5eca12ea1f7d538026d7e7d1adc (diff)
downloadbcm5719-llvm-051bdb0b22a3eba1d8eb97f071195a13559a965a.tar.gz
bcm5719-llvm-051bdb0b22a3eba1d8eb97f071195a13559a965a.zip
[PM] Fix a silly bug in my recent update to the CG update logic.
I used the wrong variable to update. This was even covered by a unittest I wrote, and the comments for the unittest were correct (if confusing) but the test itself just matched the buggy behavior. =[ llvm-svn: 307764
Diffstat (limited to 'llvm/unittests/Analysis/CGSCCPassManagerTest.cpp')
-rw-r--r--llvm/unittests/Analysis/CGSCCPassManagerTest.cpp12
1 files changed, 6 insertions, 6 deletions
diff --git a/llvm/unittests/Analysis/CGSCCPassManagerTest.cpp b/llvm/unittests/Analysis/CGSCCPassManagerTest.cpp
index 1e262683f2b..d46d9535fa4 100644
--- a/llvm/unittests/Analysis/CGSCCPassManagerTest.cpp
+++ b/llvm/unittests/Analysis/CGSCCPassManagerTest.cpp
@@ -1261,12 +1261,12 @@ TEST_F(CGSCCPassManagerTest, TestAnalysisInvalidationCGSCCUpdate) {
// split their SCCs. Then we re-run over the whole module. Then we re-run
// over three functions merged back into a single SCC, and then over the
// whole module again.
- EXPECT_EQ(6 + 2 + 6 + 3 + 6, FunctionAnalysisRuns);
+ EXPECT_EQ(6 + 3 + 6 + 3 + 6, FunctionAnalysisRuns);
- // Re run the function analysis twice over the entire module, and then re-run
- // it over the `(h3, h1, h2)` SCC due to invalidation. Then we re-un over
- // three functions merged back into a single SCC, and then over the whole
- // module.
- EXPECT_EQ(6 + 2 + 6 + 3 + 6, IndirectFunctionAnalysisRuns);
+ // Re run the function analysis over the entire module, and then re-run it
+ // over the `(h3, h1, h2)` SCC due to invalidation. Then we re-run it over
+ // the entire module, then the three functions merged back into a single SCC,
+ // and then over the whole module.
+ EXPECT_EQ(6 + 3 + 6 + 3 + 6, IndirectFunctionAnalysisRuns);
}
}
OpenPOWER on IntegriCloud