[PATCH] D19588: [Clang][BuiltIn][AVX512] Adding intrinsics fot align{d|q} and palignr instruction set

[Asaf Badouh via llvm-commits]

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

please change from setzero to undefined in the non mask versions.
LGTM after that.


================
Comment at: lib/Headers/avx512bwintrin.h:2063
@@ +2062,3 @@
+                 (__v8di) __B ,__N * 8,\
+                 (__v8di) _mm512_setzero_si512 (),\
+                 (__mmask64) -1);\
----------------
please change it to _mm512_undefined_epi32()

================
Comment at: lib/Headers/avx512vlintrin.h:7771
@@ +7770,3 @@
+              (__v4si)( __B),( __imm),\
+              (__v4si) _mm_setzero_si128 (),\
+              (__mmask8) -1);\
----------------
_mm_undefined_ps()


http://reviews.llvm.org/D19588