[PATCH] D76132: [LoopUnrollAndJam] Changed safety checks to consider more than 2-levels loop nest.

[Whitney Tsang via Phabricator via llvm-commits]

Whitney added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp:766
+
+static bool isEligibleLoopForm(const Loop &Root) {
+  // Root must have a child.
----------------
Whitney wrote:
> Meinersbur wrote:
> > Can the following be handled correctly?
> >  * Multiple exits
> > 
> I think it should, but I will write a LIT test to be sure. Only one AftBlock is allowed in the outermostloop.
Multiple exits are not allowed, and that's blocked in `partitionOuterLoopBlocks`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76132