[PATCH] D105216: [ScalarEvolution] Fix overflow in computeBECount.

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

I've glanced at this, and the refactor/new code seems very complicated to me.  My LGTM no longer applies.  I will try to take a closer look at the reasoning in the newly added parts, but I *strongly* request you look for ways to factor the code so that the reasoning is modular.  If we can split this into two or more patches, that would be my strong preference.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105216