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

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/docs/ReleaseNotes.rst:15
 
+* The LoopFlatten pass is now enabled by default.
 
----------------
this doesn't look like the right place for the notes
`Non-comprehensive list of changes in this release` section looks better


================
Comment at: llvm/lib/Transforms/Scalar/LoopFlatten.cpp:765
   SE->forgetLoop(FI.InnerLoop);
+  SE->forgetLoopDispositions();
   if (U)
----------------
this should probably have a proper IR test case and be committed separately (the committed separately part is moot now, but mentioned in case this gets reverted again at some point)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109958