[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