This is an archive of the discontinued LLVM Phabricator instance.

[Clang][Avx512][BuiltIn] Adding brodcast builtin to clang
ClosedPublic

Authored by m_zuckerman on Apr 12 2016, 5:24 AM.

Diff Detail

Event Timeline

m_zuckerman retitled this revision from to [Clang][Avx512][BuiltIn] Adding brodcast builtin to clang.
m_zuckerman updated this object.
m_zuckerman added reviewers: AsafBadouh, igorb, delena.
m_zuckerman added a subscriber: llvm-commits.
igorb added inline comments.Apr 12 2016, 6:00 AM
include/clang/Basic/BuiltinsX86.def
1981

it should be "avx512bw" ?

lib/Headers/avx512bwintrin.h
2065

could you please put casting on the same line with argument

m_zuckerman marked an inline comment as done.
m_zuckerman marked an inline comment as done.Apr 12 2016, 10:02 PM
igorb added inline comments.Apr 12 2016, 10:34 PM
lib/Headers/avx512bwintrin.h
2080

could you please put casting on the same line with argument ( the same regarding the code below)

igorb added inline comments.Apr 13 2016, 2:01 AM
lib/Headers/avx512dqintrin.h
175

could you please put casting on the same line with argument

214

could you please put casting on the same line with argument ( the same regarding the code below)

252

could you please put casting on the same line with argument

288

could you please put casting on the same line with argument

lib/Headers/avx512vldqintrin.h
170

could you please put casting on the same line with argument ( the same regarding the code below)

igorb accepted this revision.Apr 13 2016, 2:14 AM
igorb edited edge metadata.

Thanks!

LGTM

This revision is now accepted and ready to land.Apr 13 2016, 2:14 AM
This revision was automatically updated to reflect the committed changes.