[PATCH] D20562: [Clang][AVX512][BUILTIN] Adding intrinsics for set1

Asaf Badouh via cfe-commits cfe-commits at lists.llvm.org
Wed May 25 06:35:13 PDT 2016


AsafBadouh accepted this revision.
AsafBadouh added a comment.
This revision is now accepted and ready to land.

LGTM with small fix


================
Comment at: test/CodeGen/avx512f-builtins.c:341
@@ +340,3 @@
+  // CHECK: insertelement <64 x i8> {{.*}}, i32 6
+  // CHECK: insertelement <64 x i8> {{.*}}, i32 7
+  return _mm512_set1_epi8(d);
----------------
can you add
// CHECK: insertelement <64 x i8> {{.*}}, i32 63



================
Comment at: test/CodeGen/avx512f-builtins.c:356
@@ +355,3 @@
+  // CHECK: insertelement <32 x i16> {{.*}}, i32 7
+  return _mm512_set1_epi16(d);
+}
----------------
same here:
// CHECK: insertelement <32 x i16> {{.*}}, i32 31


http://reviews.llvm.org/D20562





More information about the cfe-commits mailing list