[PATCH] D58593: [ValueTracking] More accurate unsigned add/sub overflow detection

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 27 11:26:55 PST 2019


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

LGTM 
As you've experienced, we may uncover bugs in other passes with the improved analysis. To minimize risk and make debug potentially easier, I'd push the change as 2 separate commits (1 for add, 1 for sub).


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

https://reviews.llvm.org/D58593





More information about the llvm-commits mailing list