[NFC][DSE] Fix typo comment in eliminateDeadStores

[omern1]

We are re-using tryToMergePartialOverlappingStores, which requires DeadSI to dominate DeadSI.

Should be "DeadSI to dominate KillingSI" because that's what the check is for.

[llvmbot]

@llvm/pr-subscribers-llvm-transforms

Author: Nabeel Omer (omern1)

Changes

Fix comment in DSE.

---

Full diff: https://github.com/llvm/llvm-project/pull/75166.diff

1 Files Affected:

• (modified) llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp (+1-1)

diff --git a/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp b/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
index dd0a290252dae..008dcc53fd44f 100644
--- a/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
+++ b/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
@@ -2170,7 +2170,7 @@ static bool eliminateDeadStores(Function &F, AliasAnalysis &AA, MemorySSA &MSSA,
           auto *DeadSI = dyn_cast<StoreInst>(DeadI);
           auto *KillingSI = dyn_cast<StoreInst>(KillingI);
           // We are re-using tryToMergePartialOverlappingStores, which requires
-          // DeadSI to dominate DeadSI.
+          // DeadSI to dominate KillingSI.
           // TODO: implement tryToMergeParialOverlappingStores using MemorySSA.
           if (DeadSI && KillingSI && DT.dominates(DeadSI, KillingSI)) {
             if (Constant *Merged = tryToMergePartialOverlappingStores(

[RKSimon]

LGTM but please improve the commit message to explain the fix