[PATCH] D78932: [DSE,MSSA] Relax post-dom restriction for objs visible after return.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 10 03:14:16 PDT 2020


fhahn added a comment.

In D78932#2083972 <https://reviews.llvm.org/D78932#2083972>, @george.burgess.iv wrote:

> Eesh -- sorry for taking forever. :)
>
> I think my questions are answered, and seeing how this works SGTM. If we need something faster (or wanna experiment with other approaches), I think we have a solid way forward on that.
>
> That said, LGTM. Thanks again!


Awesome, thanks! I think there are 2 more patches or so to catch most outstanding cases of legacy DSE. After that I'll move to compile-time tuning.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78932





More information about the llvm-commits mailing list