Skip to content

Commit e884843

Browse files
committed
[IndVars] Add a todo to reflect a further oppurtunity identified in D69009
Nikita pointed out an oppurtunity, might as well document it in the code. llvm-svn: 375380
1 parent 8cbcd2f commit e884843

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

llvm/lib/Transforms/Scalar/IndVarSimplify.cpp

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2794,6 +2794,13 @@ bool IndVarSimplify::optimizeLoopExits(Loop *L, SCEVExpander &Rewriter) {
27942794
Changed = true;
27952795
continue;
27962796
}
2797+
2798+
// TODO: There might be another oppurtunity to leverage SCEV's reasoning
2799+
// here. If we kept track of the min of dominanting exits so far, we could
2800+
// discharge exits with EC >= MDEC. This is less powerful than the existing
2801+
// transform (since later exits aren't considered), but potentially more
2802+
// powerful for any case where SCEV can prove a >=u b, but neither a == b
2803+
// or a >u b. Such a case is not currently known.
27972804
}
27982805
return Changed;
27992806
}

0 commit comments

Comments
 (0)