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

[Asaf Badouh via llvm-commits]

AsafBadouh added inline comments.

================
Comment at: include/clang/Basic/BuiltinsX86.def:1581
@@ +1580,3 @@
+TARGET_BUILTIN(__builtin_ia32_pmovzxwq512_mask, "V8LLiV8sV8LLiUc","","avx512f")
+TARGET_BUILTIN(__builtin_ia32_pmovzxbw128_mask, "V8sV16cV8sUc","","avx512bw,avx512vl")
+TARGET_BUILTIN(__builtin_ia32_pmovzxbw256_mask, "V16sV16cV16sUs","","avx512bw,avx512vl")
----------------
can you please change the order of ATTRS to be consistent with the rest of the file?
I.e. (avx512bw,avx512vl) -> (avx512vl,avx512bw)


http://reviews.llvm.org/D16961