[PATCH] D22114: [InstCombine] extend vector select matching for non-splat constants

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 7 15:29:45 PDT 2016


eli.friedman added a comment.

Do we care at all about the number of uses of the operands?


================
Comment at: lib/Transforms/InstCombine/InstCombineAndOrXor.cpp:1601
@@ +1600,3 @@
+
+    // One element must be all ones, and the other must be all zeros.
+    if (!((match(EltC1, m_Zero()) && match(EltC2, m_AllOnes())) ||
----------------
What if one is undef?


http://reviews.llvm.org/D22114





More information about the llvm-commits mailing list