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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

Nice! I think this is a good thing.

I have checked this on some code i care about and as far as i can tell
it pretty much always fails with `Loop::isCanonical()`, e.g. because
the loops produced by OpenMP lowering are non-canonical by definition.
This is probably not a blocker, but it does mean that the true impact
of this change (aka, does it regress cases and require costmodel tuning?)
will not be apparent until later.


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

https://reviews.llvm.org/D109958