[PATCH] D20810: [Clang][Intrinsics][avx512] Continue Adding round cvt to clang

Asaf Badouh via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 1 06:41:32 PDT 2016


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

minor fixes, LGTM


================
Comment at: lib/Headers/avx512fintrin.h:3658
@@ +3657,3 @@
+						     _mm256_undefined_si256 (),\
+						     -1);\
+})
----------------
please add (__mmask16)

================
Comment at: lib/Headers/avx512fintrin.h:3669
@@ +3668,3 @@
+#define _mm512_maskz_cvt_roundps_ph( __W, __A, __I) __extension__ ({ \
+__builtin_ia32_vcvtps2ph512_mask ((__v16sf)( __A),\
+						    ( __I),\
----------------
please add casting to return type (__m256i)

same for the rest

================
Comment at: test/CodeGen/avx512f-builtins.c:3121
@@ +3120,3 @@
+    // CHECK: @llvm.x86.avx512.mask.cvttps2udq.512
+
+    return _mm512_mask_cvtt_roundps_epu32(__W, __U, __A, _MM_FROUND_CUR_DIRECTION);
----------------
remove empty line

================
Comment at: test/CodeGen/avx512f-builtins.c:3129
@@ +3128,3 @@
+    // CHECK: @llvm.x86.avx512.mask.cvttps2udq.512
+
+    return _mm512_maskz_cvtt_roundps_epu32(__U, __A, _MM_FROUND_CUR_DIRECTION);
----------------
remove empty line


http://reviews.llvm.org/D20810





More information about the cfe-commits mailing list