[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