[PATCH] D106327: [SCEV] Fix bug involving zero step and non-invariant RHS in trip count logic

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

In D106327#2889082 <https://reviews.llvm.org/D106327#2889082>, @efriedma wrote:

> Like I noted on the other review, I think you also need the `isLoopInvariant(RHS, L)` check if the stride is negative.

Can we fix one bug at a time though?  I'm happy to explore that one, but I'd prefer to do it on it's own with it's own discussion and tests.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106327