[PATCH] D89650: [DSE] Remove noop stores after killing stores for a MemoryDef.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 28 04:11:00 PDT 2020


fhahn added a comment.





================
Comment at: llvm/test/Transforms/DeadStoreElimination/MSSA/noop-stores.ll:291
 
 ; Test case from PR47887. Currently we eliminate the dead `store i32 %inc, i32* %x`,
 ; but not the no-op `store i32 %lv, i32* %x`. That is because no-op stores are
----------------
xbolva00 wrote:
> Update comment?
Thanks, I updated the comment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89650



More information about the llvm-commits mailing list