[PATCH] D72146: [DSE] Add first version of MemorySSA-backed DSE.
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 3 15:48:30 PST 2020
fhahn added a comment.
In D72146#1803699 <https://reviews.llvm.org/D72146#1803699>, @Tyker wrote:
> i haven't looked in details at your patch but i was working on something similar D72182 <https://reviews.llvm.org/D72182>.
Interesting, thanks for sharing! I had a brief look at your patch and I think it would be great if we could combine our efforts. I tried to keep the first patches relatively straight forward for easy review and add support for more cases incrementally. It seems like your patch has propagation of non-escaping object info and checks for barriers across multiple blocks. Do you think it would make sense to use D72146 <https://reviews.llvm.org/D72146> (this patch) as foundation and port the improvements from your patch? It would be great if you could have a look and let me know what you think!
There also are a few things to improve the walking in this patch: 1. deal with reaching the end of the walk, 2. handle cases with defs in multiple paths. Both should be quite straight forward to add (I have a patch for the first one nearly ready), but I think it would be great to do this in incremental steps as separate patches.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72146/new/
https://reviews.llvm.org/D72146
More information about the llvm-commits
mailing list