[PATCH] D36536: [X86][AVX512] Add combine for TESTM

[Guy Blank via Phabricator via llvm-commits]

guyblank added inline comments.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:35501
+      ISD::isBuildVectorAllZeros(Op1.getNode()))
+    return DAG.getSplatBuildVector(VT, DL, DAG.getConstant(0, DL, MVT::i8));
+
----------------
RKSimon wrote:
> Worth adding vXi1 support to the getZeroVector() helper?
It's already supported, I somehow missed that.
Thanks.


https://reviews.llvm.org/D36536