This is an archive of the discontinued LLVM Phabricator instance.

[Clang][AVX512][BuiltIn] Adding avx512 ( vcvt ) builtin to clang
ClosedPublic

Authored by m_zuckerman on Apr 9 2016, 9:05 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

m_zuckerman updated this revision to Diff 53135.Apr 9 2016, 9:05 AM
m_zuckerman retitled this revision from to [Clang][AVX512][BuiltIn] Adding avx512 ( vcvt ) builtin to clang.
m_zuckerman updated this object.
m_zuckerman added reviewers: AsafBadouh, igorb, delena.
m_zuckerman added a subscriber: llvm-commits.
delena edited edge metadata.Apr 9 2016, 9:56 AM

Please add tests where the rounding mode is not "current"

test/CodeGen/avx512f-builtins.c
2786 ↗(On Diff #53135)

Please use _CURRENT_instead of 4.

m_zuckerman updated this revision to Diff 53154.Apr 9 2016, 1:28 PM
m_zuckerman edited edge metadata.
delena accepted this revision.Apr 9 2016, 11:23 PM
delena edited edge metadata.
This revision is now accepted and ready to land.Apr 9 2016, 11:23 PM
This revision was automatically updated to reflect the committed changes.