[PATCH] D32143: [InstSimplify] use ConstantRange to simplify more and-of-icmps

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 24 08:08:58 PDT 2017


spatel added a comment.

In https://reviews.llvm.org/D32143#733869, @sanjoy wrote:

> LGTM!  I did not sanity check the test changes, I've assumed you did that. :)


Yes, I have two 10x10 graphs for the predicate combinations...there's a pattern in here that we could probably encode into the predicate enum values (we do something like that for the fcmp predicates), but I don't see that pattern yet. :)
And I spot-checked several of these using Alive for extra confidence.


https://reviews.llvm.org/D32143





More information about the llvm-commits mailing list