Correctly report modified status for DSE
authorserge-sans-paille <sguelton@redhat.com>
Thu, 4 Jun 2020 20:23:32 +0000 (22:23 +0200)
committerserge-sans-paille <sguelton@redhat.com>
Fri, 5 Jun 2020 13:59:42 +0000 (15:59 +0200)
Differential Revision: https://reviews.llvm.org/D81233

llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp

index 36d8b4b..bf5ba72 100644 (file)
@@ -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 @@ public:
       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();