[PATCH] D117926: [SLP] Optionally preserve MemorySSA
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 27 13:52:19 PST 2022
reames planned changes to this revision.
reames added a comment.
I am setting this aside for the moment. The consensus of review seems to be that landing a working but slow preservation and then speeding it up incrementally is not desired. I think that's a poor choice, but am willing to comply.
I will refresh this patch once I have the full complexity of a working and fast MSSA preservation.
This will likely be a while as in the delay caused by the review, I've found a non-mssa based fix for the original compile time issue and will be proceeding with that one instead. I hope to get back to the MSSA update because I think that is generally worthwhile, but it's no longer a priority for me, and I'm currently very short on time.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117926/new/
https://reviews.llvm.org/D117926
More information about the llvm-commits
mailing list