This is an archive of the discontinued LLVM Phabricator instance.

[Clang][AVX512][BuiltIn] Adding intrinsics of VGATHER{DPS|DPD} , VPGATHER{QD|QQ|DD|DQ} and VGATHERPF{0|1}{DPS|QPS|DPD|QPD} instruction set .
ClosedPublic

Authored by m_zuckerman on Apr 18 2016, 7:44 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

m_zuckerman retitled this revision from to [Clang][AVX512][BuiltIn] Adding intrinsics of VGATHER{DPS|DPD} , VPGATHER{QD|QQ|DD|DQ} and VGATHERPF0{DPS|QPS|DPD|QPD} instruction set . .
m_zuckerman updated this object.
m_zuckerman added reviewers: AsafBadouh, delena, igorb.
m_zuckerman added a subscriber: llvm-commits.
m_zuckerman retitled this revision from [Clang][AVX512][BuiltIn] Adding intrinsics of VGATHER{DPS|DPD} , VPGATHER{QD|QQ|DD|DQ} and VGATHERPF0{DPS|QPS|DPD|QPD} instruction set . to [Clang][AVX512][BuiltIn] Adding intrinsics of VGATHER{DPS|DPD} , VPGATHER{QD|QQ|DD|DQ} and VGATHERPF{0|1}{DPS|QPS|DPD|QPD} instruction set . .Apr 20 2016, 2:08 AM
igorb added inline comments.Apr 21 2016, 12:07 AM
include/clang/Basic/BuiltinsX86.def
1083 ↗(On Diff #54067)

could you please move new TARGET_BUILTIN before prefetch , so all prefetch buildins remain in one group.

lib/Headers/avx512pfintrin.h
29 ↗(On Diff #54067)

please change to __AVX512PFINTRIN_H

igorb accepted this revision.Apr 21 2016, 4:59 AM
igorb edited edge metadata.

LGTM

This revision is now accepted and ready to land.Apr 21 2016, 4:59 AM
This revision was automatically updated to reflect the committed changes.