[PATCH] D33543: Re-enable "[SCEV] Do not fold dominated SCEVUnknown into AddRecExpr start"

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 25 20:05:52 PDT 2017


mkazantsev added a comment.

Sanjoy, what happens is nearly following: without the change, LSR was making an overconfident simplification basing on a wrong SCEV. Apparently it did not need the IV analysis to do this. With the change, it chose a different way to simplify (that wasn't so confident), and this way required the IV analysis.


https://reviews.llvm.org/D33543





More information about the llvm-commits mailing list