This is an archive of the discontinued LLVM Phabricator instance.

[Clang][Builtin][AVX512] Adding intrinsics fot cvt{u}si2s{d|s} cvt{sd|ss}2{ss|sd} instruction set
ClosedPublic

Authored by m_zuckerman on Apr 30 2016, 8:21 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

m_zuckerman retitled this revision from to [Clang][Builtin][AVX512] Adding intrinsics fot cvt{sd|ss} instruction set .
m_zuckerman updated this object.
m_zuckerman added reviewers: AsafBadouh, igorb, delena.
m_zuckerman retitled this revision from [Clang][Builtin][AVX512] Adding intrinsics fot cvt{sd|ss} instruction set to [Clang][Builtin][AVX512] Adding intrinsics fot cvt{u}si2s{d|s} cvt{sd|ss}2{ss|sd} instruction set .
m_zuckerman added a subscriber: llvm-commits.
AsafBadouh added inline comments.May 2 2016, 7:52 AM
include/clang/Basic/BuiltinsX86.def
1946 ↗(On Diff #55717)

ULLi -> LLi (remove the unsigned)

1948 ↗(On Diff #55717)

ULLi -> LLi (remove the unsigned)

1950 ↗(On Diff #55717)

the second operand is unsigned int, i->Ui

1951 ↗(On Diff #55717)

the second operand of the builtin is ULLi (for unsinged long long)

1952 ↗(On Diff #55717)

the second operand is unsigned int, i->Ui

1953 ↗(On Diff #55717)

same, d-> ULLi

m_zuckerman updated this revision to Diff 55997.May 3 2016, 8:03 AM
m_zuckerman marked 6 inline comments as done.
m_zuckerman updated this revision to Diff 56001.May 3 2016, 8:16 AM
AsafBadouh accepted this revision.May 3 2016, 8:36 AM
AsafBadouh edited edge metadata.

LGTM

This revision is now accepted and ready to land.May 3 2016, 8:36 AM
This revision was automatically updated to reflect the committed changes.