This is an archive of the discontinued LLVM Phabricator instance.

[CLANG] [AVX512] [BUILTIN] Adding pmovsx{b|d|w}{w|d|q}{128|256|512} builtin to clang
ClosedPublic

Authored by m_zuckerman on Feb 7 2016, 3:39 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

m_zuckerman updated this revision to Diff 47123.Feb 7 2016, 3:39 AM
m_zuckerman retitled this revision from to [CLANG] [AVX512] [BUILTIN] Adding pmovsx{b|d|w}{w|d|q}{128|256|512} builtin to clang .
m_zuckerman updated this object.
m_zuckerman added reviewers: delena, AsafBadouh, igorb.
m_zuckerman added a subscriber: llvm-commits.
igorb accepted this revision.Feb 9 2016, 1:31 AM
igorb edited edge metadata.

minor comment in code,
LGTM

include/clang/Basic/BuiltinsX86.def
1575 ↗(On Diff #47123)

please add space after comma

This revision is now accepted and ready to land.Feb 9 2016, 1:31 AM
This revision was automatically updated to reflect the committed changes.