[PATCH] D16955: [CLANG] [AVX512] [BUILTIN] Adding pmovsx{b|d|w}{w|d|q}{128|256|512} builtin to clang

[Igor Breger via llvm-commits]

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

minor comment in code, 
LGTM


================
Comment at: include/clang/Basic/BuiltinsX86.def:1575
@@ -1574,1 +1574,3 @@
 TARGET_BUILTIN(__builtin_ia32_reduceps512_mask, "V16fV16fIiV16fUsIi", "", "avx512dq")
+TARGET_BUILTIN(__builtin_ia32_pmovsxbw512_mask, "V32sV32cV32sUi","","avx512bw")
+TARGET_BUILTIN(__builtin_ia32_pmovsxbd512_mask, "V16iV16cV16iUs","","avx512f")
----------------
please add space after comma


http://reviews.llvm.org/D16955