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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits