[PATCH] D104679: [LoopUnrolling] Add flag to restrict the unroll with large loop size

[Nikita Popov via Phabricator via llvm-commits]

nikic added reviewers: reames, nikic.
nikic added a comment.

Skipping very large loops might make sense as a compile-time optimization (don't bother analyzing loops that definitely can't be unrolled), but I don't think it's a solution to a stack overflow in SCEV. It can happen through other ways than loop unrolling. SCEV uses recursive construction and stack overflows are a known issue -- it deals with them through various depth cutoffs. Maybe one is missing somewhere?


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

https://reviews.llvm.org/D104679