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

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a reviewer: RKSimon.
RKSimon added inline comments.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:35497
+  // TEST op0, BUILD_VECTOR(all_zero) -> BUILD_VECTOR(all_zero)
+  // TEST BUILD_VECTOR(all_zero) -> BUILD_VECTOR(all_zero)
+
----------------
```
// TEST BUILD_VECTOR(all_zero), op1 -> BUILD_VECTOR(all_zero)
```


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:35501
+      ISD::isBuildVectorAllZeros(Op1.getNode()))
+    return DAG.getSplatBuildVector(VT, DL, DAG.getConstant(0, DL, MVT::i8));
+
----------------
Worth adding vXi1 support to the getZeroVector() helper?


https://reviews.llvm.org/D36536