This is an archive of the discontinued LLVM Phabricator instance.

[CLANG][AVX512][BUILTIN] Completing and Adding scalar Intrinsics to clang
AbandonedPublic

Authored by m_zuckerman on Apr 7 2016, 7:41 AM.

Details

Summary

rsqrt14s{s|d}
rcp14s{s|d}
getexps{s|d}
getmants{s|d}
sqrts{s|d}
fmadd_s{s|d}
fmadd_round_s{s|d}
fmsub_s{s|d}
fmsub_round_s{s|d}
fnmadd_s{s|d}
fnmadd_round_s{s|d}
fnmsub_s{s|d}
fnmsub_round_s{s|d}

Diff Detail

Event Timeline

m_zuckerman updated this revision to Diff 52916.Apr 7 2016, 7:41 AM
m_zuckerman retitled this revision from to [CLANG][AVX512][BUILTIN] Completing and Adding scalar Intrinsics to clang .
m_zuckerman updated this object.
m_zuckerman added reviewers: AsafBadouh, igorb, delena.
m_zuckerman added a subscriber: llvm-commits.
delena edited edge metadata.Apr 12 2016, 2:01 AM

you mixed all together. Please add fmaddss, fmadsd only.

AsafBadouh added inline comments.Apr 12 2016, 4:18 AM
lib/Headers/avx512fintrin.h
4356

I think it should be (v4sf) (-(B))

4364

same
I think it should be (v4sf) (-(B))

4409

please add round parentheses to __A

4479

some "-" are missing

4571

some "-" is missing

4579

some "-" is missing

4660

I'm not sure but I think it should be:
(v2df) (A),
(v2df) -(B),
(v2df) -(W),

m_zuckerman abandoned this revision.Apr 20 2016, 8:36 AM
m_zuckerman marked 6 inline comments as done.

This review was split into two reviews:

The first, [Clang][AVX512][BUILTIN] Adding scalar intrinsics for rsqrt14 ,rcp14, getexp and getmant instruction set
http://reviews.llvm.org/D19326
The second, [Clang][AVX512][Builtin] adding intrinsics for vf{n}madd{ss|sd} and vf{n}sub{ss|sd} instruction set
http://reviews.llvm.org/D19320