This is an archive of the discontinued LLVM Phabricator instance.

[Clang][AVX512][Intrinsics] Adding two definitions _mm512_setzero and _mm512_setzero_epi32
ClosedPublic

Authored by m_zuckerman on Jun 1 2016, 9:36 AM.

Diff Detail

Event Timeline

m_zuckerman updated this revision to Diff 59237.Jun 1 2016, 9:36 AM
m_zuckerman retitled this revision from to [Clang][AVX512][Intrinsics] Adding two definitions _mm512_setzero and _mm512_setzero_epi32.
m_zuckerman updated this object.
m_zuckerman added reviewers: AsafBadouh, igorb, delena.
m_zuckerman added a subscriber: cfe-commits.
delena edited edge metadata.Jun 1 2016, 12:28 PM

Why do you need them. Is it a part of API?

m_zuckerman updated this revision to Diff 59389.Jun 2 2016, 8:45 AM
m_zuckerman edited edge metadata.

Can you add tests for the existing _mm512_setzero_* intrinsics as well please?

m_zuckerman updated this revision to Diff 59659.Jun 5 2016, 4:25 AM
RKSimon accepted this revision.Jun 5 2016, 6:46 AM
RKSimon edited edge metadata.

LGTM - add test_mm512_setzero_pd() as well if you can.

test/CodeGen/avx512f-builtins.c
24

__m512d test_mm512_setzero_pd() ?

This revision is now accepted and ready to land.Jun 5 2016, 6:46 AM
This revision was automatically updated to reflect the committed changes.