This revision was automatically updated to reflect the committed changes.
Closed by commit rL271832: [Clang][AVX512][Intrinsics] Adding two definitions
_mm512_setzero and… (authored by mzuckerm).
Changed prior to commit:
http://reviews.llvm.org/D20871?vs=59659=59665#toc
Repository:
rL LLVM
RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.
LGTM - add test_mm512_setzero_pd() as well if you can.
Comment at: test/CodeGen/avx512f-builtins.c:7291
@@ +7290,3 @@
+
+__m512i test_mm512_setzero_ps()
+{
m_zuckerman updated this revision to Diff 59659.
http://reviews.llvm.org/D20871
Files:
lib/Headers/avx512fintrin.h
test/CodeGen/avx512f-builtins.c
Index: test/CodeGen/avx512f-builtins.c
===
--- test/CodeGen/avx512f-builtins.c
RKSimon added a comment.
Can you add tests for the existing _mm512_setzero_* intrinsics as well please?
http://reviews.llvm.org/D20871
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
m_zuckerman updated this revision to Diff 59389.
http://reviews.llvm.org/D20871
Files:
lib/Headers/avx512fintrin.h
test/CodeGen/avx512f-builtins.c
Index: test/CodeGen/avx512f-builtins.c
===
--- test/CodeGen/avx512f-builtins.c
m_zuckerman added a comment.
Yes its API
http://reviews.llvm.org/D20871
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
RKSimon added a subscriber: RKSimon.
RKSimon added a comment.
Tests?
http://reviews.llvm.org/D20871
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
delena added a comment.
Why do you need them. Is it a part of API?
http://reviews.llvm.org/D20871
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
m_zuckerman created this revision.
m_zuckerman added reviewers: AsafBadouh, igorb, delena.
m_zuckerman added a subscriber: cfe-commits.
http://reviews.llvm.org/D20871
Files:
lib/Headers/avx512fintrin.h
Index: lib/Headers/avx512fintrin.h