[PATCH] D103991: [SCEV] Move mustprogress based no-self-wrap logic so it applies to all exit conditions

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

In D103991#3142766 <https://reviews.llvm.org/D103991#3142766>, @nikic wrote:

> Also no difference. I suspect that we're seeing some kind of second order effect here, not an issue in the code itself.

I agree and am going to revert the two speculative changes as they just complicate the code.

I don't really know what else to do here on the original regression.  It sure seems like inferring flags is simply making some other piece of code slower.  As things stand, I plan to leave the original code in tree as the regression is small, but I find that result unsatisfying.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103991