[PATCH] D25962: [EarlyCSE] Enable MemorySSA for middle-end run of EarlyCSE.

[Geoff Berry via Phabricator via llvm-commits]

gberry added a subscriber: sebpop.
gberry added a comment.

In https://reviews.llvm.org/D25962#610123, @reames wrote:

> This seems like a reasonable tradeoff, but I'd really like to hear from someone more familiar with the general status of MemorySSA.  Is it ready to be on by default?


I haven't gotten a change to do this yet, but one thing I'd like to try out is moving GVNHoist to just after this pass of EarlyCSE, which should eliminate one of the MemorySSA construction passes.  I discussed this with @sebpop and he seemed amenable.


https://reviews.llvm.org/D25962