llvm.org GIT mirror llvm / 3870ce2
[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. =[ git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@307764 91177308-0d34-0410-b5e6-96231b3b80d8 Chandler Carruth 3 years ago
2 changed file(s) with 10 addition(s) and 10 deletion(s). Raw diff Collapse all Expand all
372372
373373 // Ensure new SCCs' function analyses are updated.
374374 if (NeedFAMProxy)
375 updateNewSCCFunctionAnalyses(*C, G, AM);
375 updateNewSCCFunctionAnalyses(NewC, G, AM);
376376
377377 // Also propagate a normal invalidation to the new SCC as only the current
378378 // will get one from the pass manager infrastructure.
12601260 // split their SCCs. Then we re-run over the whole module. Then we re-run
12611261 // over three functions merged back into a single SCC, and then over the
12621262 // whole module again.
1263 EXPECT_EQ(6 + 2 + 6 + 3 + 6, FunctionAnalysisRuns);
1264
1265 // Re run the function analysis twice over the entire module, and then re-run
1266 // it over the `(h3, h1, h2)` SCC due to invalidation. Then we re-un over
1267 // three functions merged back into a single SCC, and then over the whole
1268 // module.
1269 EXPECT_EQ(6 + 2 + 6 + 3 + 6, IndirectFunctionAnalysisRuns);
1270 }
1271 }
1263 EXPECT_EQ(6 + 3 + 6 + 3 + 6, FunctionAnalysisRuns);
1264
1265 // Re run the function analysis over the entire module, and then re-run it
1266 // over the `(h3, h1, h2)` SCC due to invalidation. Then we re-run it over
1267 // the entire module, then the three functions merged back into a single SCC,
1268 // and then over the whole module.
1269 EXPECT_EQ(6 + 3 + 6 + 3 + 6, IndirectFunctionAnalysisRuns);
1270 }
1271 }