Skip to content

Commit

Permalink
[DSE] Drop redundant WalkerStepLimit adjustment
Browse files Browse the repository at this point in the history
There is a general WalkerStepLimit adjustment higher up in the
loop, and I don't see any reason why this particular case would
need additional adjustment. Furthermore, this could underflow.
  • Loading branch information
nikic committed Mar 3, 2022
1 parent 44a4303 commit b214f55
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1313,7 +1313,6 @@ struct DSEState {
// memory location and not located in different loops.
if (!isGuaranteedLoopIndependent(CurrentI, KillingI, *CurrentLoc)) {
LLVM_DEBUG(dbgs() << " ... not guaranteed loop independent\n");
WalkerStepLimit -= 1;
CanOptimize = false;
continue;
}
Expand Down

0 comments on commit b214f55

Please sign in to comment.