[PATCH] D18929: [Clang][AVX512][BuiltIn] Adding avx512 ( punpck{h|l}{dq|qdq}{128|256|512}, rndscale{ss|sd}, {scalef{ss|sd|pd512|ps512} ) builtin to clang ....

[Elena Demikhovsky via llvm-commits]

delena added a comment.

Please add tests with non-default round mode.


================
Comment at: test/CodeGen/avx512f-builtins.c:2660
@@ +2659,3 @@
+  // CHECK: @llvm.x86.avx512.mask.rndscale.sd
+  return _mm_roundscale_sd(__A, __B, 4); 
+}
----------------
use "CURRENT", please

================
Comment at: test/CodeGen/avx512f-builtins.c:2665
@@ +2664,3 @@
+  // CHECK: @llvm.x86.avx512.mask.rndscale.sd
+    return _mm_mask_roundscale_sd(__W,__U,__A,__B,3);
+}
----------------
Why 3?

================
Comment at: test/CodeGen/avx512f-builtins.c:2695
@@ +2694,3 @@
+
+}
+
----------------
???


http://reviews.llvm.org/D18929