[PATCH] D109958: [LoopFlatten] Enable it by default

[Alan Phipps via Phabricator via llvm-commits]

alanphipps added a comment.

In D109958#3056240 <https://reviews.llvm.org/D109958#3056240>, @SjoerdMeijer wrote:

> I assume you have a downstream change to enable it by default, just like us. So if we are going to enable this by default under the NPM, I am not sure if anything will change for you

Yes, fair enough -- rereading the discussion, I was concerned that the MemorySSA issue for the LPM wasn't well understood and perhaps something was being overlooked in the implementation, but it sounds like the concern is more pertinent to performance impact/fluctuation on the LPM and not the implementation itself, which is complete.  I think we are good enabling it downstream.


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

https://reviews.llvm.org/D109958