[PATCH] D33585: [InstSimplify] Use commutable matchers to shorten some code

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 26 10:18:15 PDT 2017


spatel added a comment.

In https://reviews.llvm.org/D33585#765865, @craig.topper wrote:

> Also should the matcher for the ands be commutable too?


If we can make that work too, that would be great. :)
I don't have a preference about whether we change this in one step or more. Whatever seems easier to you. As long as we have at least a couple more tests, we should be sure that it all works as expected.


https://reviews.llvm.org/D33585





More information about the llvm-commits mailing list