[llvm] 4245100 - Correctly report modified status for DSE

[via llvm-commits]

Author: serge-sans-paille
Date: 2020-06-05T15:59:42+02:00
New Revision: 424510095d4d4fabcb1aa6526a0e78b98516e175

URL: https://github.com/llvm/llvm-project/commit/424510095d4d4fabcb1aa6526a0e78b98516e175
DIFF: https://github.com/llvm/llvm-project/commit/424510095d4d4fabcb1aa6526a0e78b98516e175.diff

LOG: Correctly report modified status for DSE

Differential Revision: https://reviews.llvm.org/D81233

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 36d8b4b06cf4..bf5ba7294d86 100644
--- a/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
+++ b/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
@@ -1859,6 +1859,7 @@ bool eliminateDeadStoresMemorySSA(Function &F, AliasAnalysis &AA,
         LLVM_DEBUG(dbgs() << "DSE: Remove Dead Store:\n  DEAD: " << *SI
                           << '\n');
         NumNoopStores++;
+        MadeChange = true;
         continue;
       }
     }
@@ -2051,7 +2052,7 @@ class DSELegacyPass : public FunctionPass {
       PostDominatorTree &PDT =
           getAnalysis<PostDominatorTreeWrapperPass>().getPostDomTree();
 
-      eliminateDeadStoresMemorySSA(F, AA, MSSA, DT, PDT, TLI);
+      Changed = eliminateDeadStoresMemorySSA(F, AA, MSSA, DT, PDT, TLI);
     } else {
       MemoryDependenceResults &MD =
           getAnalysis<MemoryDependenceWrapperPass>().getMemDep();