[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: test/CodeGen/avx512f-builtins.c:6231
+  // CHECK: bitcast <16 x i1> %{{.*}} to i16
+  // CHECK: and i32 %{{.*}}, 255
+  // CHECK: shl i32 %{{.*}}, 8
----------------
Does this really produce kunpackb in the backend? The type promotion here makes me skeptic


================
Comment at: test/CodeGen/avx512f-builtins.c:6235
+  // CHECK: bitcast i16 %{{.*}} to <16 x i1>
+  return _mm512_mask_cmpneq_epu32_mask(_mm512_kunpackb(_mm512_cmpneq_epu32_mask(__A,__B),_mm512_cmpneq_epu32_mask(__C,__D)),__E, __F);
 }
----------------
Please add some line breaks to this.


https://reviews.llvm.org/D39719