Skip to content

Commit 1a55fbc

Browse files
committed
[DSE,MemorySSA] Use NumRedundantStores instead of NumNoopStores.
Legacy DSE uses NumRedundantStores, while MemorySSA DSE uses NumNoopStores. We should just use the same counter.
1 parent fdf71d4 commit 1a55fbc

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,6 @@ STATISTIC(NumFastStores, "Number of stores deleted");
8484
STATISTIC(NumFastOther, "Number of other instrs removed");
8585
STATISTIC(NumCompletePartials, "Number of stores dead by later partials");
8686
STATISTIC(NumModifiedStores, "Number of stores modified");
87-
STATISTIC(NumNoopStores, "Number of noop stores deleted");
8887
STATISTIC(NumCFGChecks, "Number of stores modified");
8988
STATISTIC(NumCFGTries, "Number of stores modified");
9089
STATISTIC(NumCFGSuccess, "Number of stores modified");
@@ -2158,7 +2157,7 @@ bool eliminateDeadStoresMemorySSA(Function &F, AliasAnalysis &AA,
21582157
if (isRemovable(SI) && State.storeIsNoop(KillingDef, SILoc, SILocUnd)) {
21592158
LLVM_DEBUG(dbgs() << "DSE: Remove No-Op Store:\n DEAD: " << *SI << '\n');
21602159
State.deleteDeadInstruction(SI);
2161-
NumNoopStores++;
2160+
NumRedundantStores++;
21622161
MadeChange = true;
21632162
continue;
21642163
}

0 commit comments

Comments
 (0)