[llvm] b214f55 - [DSE] Drop redundant WalkerStepLimit adjustment
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 3 00:42:46 PST 2022
Author: Nikita Popov
Date: 2022-03-03T09:42:38+01:00
New Revision: b214f550f7ef679ce13011295a03f47899c56978
URL: https://github.com/llvm/llvm-project/commit/b214f550f7ef679ce13011295a03f47899c56978
DIFF: https://github.com/llvm/llvm-project/commit/b214f550f7ef679ce13011295a03f47899c56978.diff
LOG: [DSE] Drop redundant WalkerStepLimit adjustment
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.
Added:
Modified:
llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp b/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
index 0b7ebf0932359..fd6c6014b45de 100644
--- a/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
+++ b/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
@@ -1313,7 +1313,6 @@ struct DSEState {
// memory location and not located in
diff erent loops.
if (!isGuaranteedLoopIndependent(CurrentI, KillingI, *CurrentLoc)) {
LLVM_DEBUG(dbgs() << " ... not guaranteed loop independent\n");
- WalkerStepLimit -= 1;
CanOptimize = false;
continue;
}
More information about the llvm-commits
mailing list