[PATCH] D112321: [DSE] Support redundant stores eliminated by memset.

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 23 06:30:39 PDT 2021


xbolva00 added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp:1998
+                               UpperInst](MemoryLocation UpperLoc) {
+        if (DefInst->isIdenticalTo(UpperInst))
+          return true;
----------------
Shouldnt this condition also handle pr50339 testcase in stores-of-existing-values.ll?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112321/new/

https://reviews.llvm.org/D112321



More information about the llvm-commits mailing list