[PATCH] D19821: [EarlyCSE] Port to use MemorySSA (disabled by default). NFC.

[Geoff Berry via llvm-commits]

gberry added a comment.

@reames @dberlin Regarding the compile time impact, do you think it would be worth pursuing a change to make EarlyCSE's use of MemorySSA optional?  That way we could avoid using it for early passes EarlyCSE and only use it for later ones, perhaps even influenced by optimization level?  A related aspect of the plan for MemorySSA that I'd like to understand is how well we think we'll be able to amortize the cost of building it by preserving/maintaining it across passes.  Daniel, can you share your thoughts on that?


http://reviews.llvm.org/D19821