[PATCH] Exit ScalarEvolution::getMulExpr Early when Choose Overflows
Andrew Trick
atrick at apple.com
Fri Aug 29 17:30:16 PDT 2014
I know the problem with the loop structure isn't a problem with your change per-say, but it makes your change a lot harder to understand.
It seems obvious to me that the 2nd and 3rd loop are redundant. But I have been wrong before. Can you confirm? If you agree, would you be willing to remove that extra loop as a separate checkin before committing your change?
http://reviews.llvm.org/D5113
More information about the llvm-commits
mailing list