[PATCH] D117926: [SLP] Optionally preserve MemorySSA

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

Ok, was able to spot the additional construction time.  It took about 15 ms.

For context, the original example spends about 3,23 seconds in SLP w/o MSSA, and the (horribly unoptimized) preservation currently takes an additional 5.25 seconds on top of that.

Quite literally, different orders of magnitudes.  If we can get SSA preservation down to something reasonable - again, incrementalism please - I'd argue using it here is entirely reasonable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117926