[PATCH] D82588: [DSE] Look through memory PHI arguments when removing noop stores in MSSA.

[Zoe Carver via Phabricator via llvm-commits]

zoecarver created this revision.
zoecarver added reviewers: fhahn, jfb, asbirlea.
zoecarver added a project: LLVM.
Herald added a subscriber: george.burgess.iv.
zoecarver marked an inline comment as done.
zoecarver added inline comments.


================
Comment at: llvm/test/Transforms/DeadStoreElimination/MSSA/noop-stores.ll:96
 bb1:
   br label %bb3
 bb2:
----------------
I'm going to add a few negative tests.


Adds support for "following" memory through MSSA PHI arguments. This will help catch more noop stores that exist between blocks.

Originally part of D79391 <https://reviews.llvm.org/D79391>.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D82588

Files:
  llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
  llvm/test/Transforms/DeadStoreElimination/MSSA/noop-stores.ll
  llvm/test/Transforms/DeadStoreElimination/MSSA/simple-todo.ll
  llvm/test/Transforms/DeadStoreElimination/MSSA/simple.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D82588.273462.patch
Type: text/x-patch
Size: 5900 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200625/8ace6800/attachment-0001.bin>