[llvm] b214f55 - [DSE] Drop redundant WalkerStepLimit adjustment

[Nikita Popov via llvm-commits]

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;
       }