[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