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

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

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 :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82407