[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