[PATCH] D38531: Improve clamp recognition in ValueTracking.

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 16 14:54:37 PDT 2017


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

LGTM.

If you want to change that first equality predicate check (the one that was removed from the top of the function) into an assert, I think that would be good since it's not obvious that we've filtered those cases out before we call this function.


https://reviews.llvm.org/D38531





More information about the llvm-commits mailing list