[PATCH] D28629: [SCEV] Limit recursion depth of constant evolving

[Andrew Trick via Phabricator via llvm-commits]

atrick accepted this revision.
atrick added a comment.
This revision is now accepted and ready to land.

Sounds good to me. I don't think you need to add a command line flag.

FWIW, I'm also in favor of moving to iterative solutions, although that does not always eliminate the need for arbitrary thresholds to bound the compute time.

I'm strongly against unbounded recursion anywhere in the compiler.


Repository:
  rL LLVM

https://reviews.llvm.org/D28629