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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to