[PATCH] D16985: [CLANG] [AVX512] [BUILTIN] Adding prol{d|q|w}{128|256|512} builtin to clang .

[Elena Demikhovsky via llvm-commits]

delena added inline comments.

================
Comment at: lib/Headers/avx512vlintrin.h:35
@@ -34,1 +34,3 @@
 /* Integer compare */
+static  __inline __v2di __DEFAULT_FN_ATTRS
+_mm_setzero_di(void) {
----------------
The comment should be moved down - int compare is the next one.

================
Comment at: lib/Headers/avx512vlintrin.h:4608
@@ -4602,1 +4607,3 @@
 
+#define _mm_rol_epi32(a, b) __extension__ ({\
+   (__m128i)__builtin_ia32_prold128_mask((__v4si) (a), (b),\
----------------
I thought we are adding compiler intrinsics as static inline functions. 


http://reviews.llvm.org/D16985