[PATCH] D111727: [DSE] Eliminates dead store of an exisiting value (PR16520)

[Dawid Jurczak via Phabricator via llvm-commits]

yurai007 marked 5 inline comments as done.
yurai007 added a comment.

In D111727#3061936 <https://reviews.llvm.org/D111727#3061936>, @lebedev.ri wrote:

> All comments from the original review still apply.

Do you have something specific in mind? It looks like all comments from old review are done (even if not marked as "fixed" which is quite confusing) except one from Nikita addressed in this change.



================
Comment at: llvm/test/Transforms/DeadStoreElimination/stores-of-existing-values.ll:310
   ret void
 }
----------------
xbolva00 wrote:
> Is this top of trunk?
Oops, sorry. Now should be fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111727