[PATCH] D24419: [InstCombine] use commutative matchers for patterns with commutative operators

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 22 14:36:47 PDT 2016


sanjoy added a subscriber: sanjoy.
sanjoy added a comment.

Have you considered treating this as a canonicalization problem?  That is, always canonicalize `(X & A) | (A ^ Y)` to `(A & X) | (A ^ Y)` (for all commutative ops)?


https://reviews.llvm.org/D24419





More information about the llvm-commits mailing list