[PATCH] D99929: [SCEV] Fix false-positive recognition of simple recurrences. PR49856

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 6 19:30:04 PDT 2021


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

I posted an alternate fix here: https://reviews.llvm.org/D100004.  Let me know what you think.  Stylistically, I prefer that one, but yours appears to be correct as well.

Given that, I'm LGTMing this, but asking you not to land unless you spot a problem with the one I linked to.  If you do, feel free to land this, and I can work on top of it.  I don't want to delay the correctness fix over a debatable style issue.


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

https://reviews.llvm.org/D99929



More information about the llvm-commits mailing list