Skip to content

Commit e4ea2d5

Browse files
committed
[StructurizeCFG] Correctly depend on UniformityAnalysis
Small oversight in https://reviews.llvm.org/D145688 - the pass' dependency was not updated to reflect the change to UA. Also, change DivergenceAnalysis to UniformityAnalysis in a comment. That way, StructurizeCFG only refers to UA and not DA anymore.
1 parent 0022b58 commit e4ea2d5

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

llvm/lib/Transforms/Scalar/StructurizeCFG.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -367,7 +367,7 @@ char StructurizeCFGLegacyPass::ID = 0;
367367

368368
INITIALIZE_PASS_BEGIN(StructurizeCFGLegacyPass, "structurizecfg",
369369
"Structurize the CFG", false, false)
370-
INITIALIZE_PASS_DEPENDENCY(LegacyDivergenceAnalysis)
370+
INITIALIZE_PASS_DEPENDENCY(UniformityInfoWrapperPass)
371371
INITIALIZE_PASS_DEPENDENCY(LowerSwitchLegacyPass)
372372
INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass)
373373
INITIALIZE_PASS_DEPENDENCY(RegionInfoPass)
@@ -1085,7 +1085,7 @@ static bool hasOnlyUniformBranches(Region *R, unsigned UniformMDKindID,
10851085
<< " has uniform terminator\n");
10861086
} else {
10871087
// Explicitly refuse to treat regions as uniform if they have non-uniform
1088-
// subregions. We cannot rely on DivergenceAnalysis for branches in
1088+
// subregions. We cannot rely on UniformityAnalysis for branches in
10891089
// subregions because those branches may have been removed and re-created,
10901090
// so we look for our metadata instead.
10911091
//

0 commit comments

Comments
 (0)