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

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

Agree with others, this seems like the wrong approach.

And yes, if we need to add a depth parameter to getSCEV, we should.  I really doubt we do though.  We probably *do* need to add one to all of the getXExpr variants though.


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

https://reviews.llvm.org/D104679