[PATCH] D82407: [DSE,MSSA] Remove dead stores to stack-allocated locations

[Whitney Tsang via Phabricator via llvm-commits]

Whitney abandoned this revision.
Whitney added a comment.

In D82407#2110262 <https://reviews.llvm.org/D82407#2110262>, @fhahn wrote:

> Thanks for working on this. I think a patch I've been working on for a while solves the same problem, but in a slightly more general way I think: D72631 <https://reviews.llvm.org/D72631>.
>
> There's one more slightly larger outstanding patch to support 'memory terminators' like lifetime.end/free as well: D72410 <https://reviews.llvm.org/D72410>. I think those patches should get us to almost cover all cases legacy DSE does, but there is plenty of potential for further improvements. I would be more than happy to sync up to avoid duplicating work, if you are also interested in working on MemorySSA backed DSE :)


@fhahn Cool, cancelled this revision. Positive side is I understand better of how MemorySSA backed DSE works after working on this patch :)
Do you know if there are plans on eliminating dead stores to array (in a loop)?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82407