[PATCH] D11212: [SCEV] Apply NSW and NUW flags via poison value analysis

Andrew Trick atrick at apple.com
Mon Jul 27 00:44:19 PDT 2015


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

Looks good. Nice comments.

For the goal of reassociating array address computation, I would have tried a different approach, but I think this is still valid and generally adds more power to SCEV.

I trust that Sanjoy has tried to poke enough holes in the logic. Overall, it looks pretty solid to me. Thanks for the thorough review, Sanjoy.


http://reviews.llvm.org/D11212







More information about the llvm-commits mailing list