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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 19 08:11:19 PDT 2021


fhahn added a comment.

> This achieves the goal by looping through all memory definitions in the current state & each definitions uses.

Does the patch description need updating? Looks like it refers to looping over all uses, but the patch here looks at the defining access of each memdef?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111727



More information about the llvm-commits mailing list