This is an archive of the discontinued LLVM Phabricator instance.

[X86][AVX512VLDQ] add reduce/range/cvt intrinsics
ClosedPublic

Authored by AsafBadouh on Jul 29 2015, 8:44 AM.

Diff Detail

Event Timeline

AsafBadouh updated this revision to Diff 30912.Jul 29 2015, 8:44 AM
AsafBadouh retitled this revision from to [X86][AVX512VLDQ] add reduce/range/cvt intrinsics .
AsafBadouh updated this object.
AsafBadouh added reviewers: mkuper, gbenyei, igorb.
AsafBadouh added a subscriber: llvm-commits.
mkuper added inline comments.Aug 2 2015, 1:06 AM
lib/Headers/avx512vldqintrin.h
855

Why are these macros and not functions?

AsafBadouh added inline comments.Aug 2 2015, 4:55 AM
lib/Headers/avx512vldqintrin.h
855

these intrinsics include imm argument so it must be macros

mkuper accepted this revision.Aug 2 2015, 5:36 AM
mkuper edited edge metadata.

LGTM

This revision is now accepted and ready to land.Aug 2 2015, 5:36 AM
This revision was automatically updated to reflect the committed changes.