This is an archive of the discontinued LLVM Phabricator instance.

[AVX512] adding AVXVBMI feature flag
ClosedPublic

Authored by m_zuckerman on Jan 14 2016, 8:13 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

m_zuckerman retitled this revision from to [AVX512] adding AVXVBMI feature flag.
m_zuckerman updated this object.
m_zuckerman added reviewers: delena, igorb, AsafBadouh.
m_zuckerman added a subscriber: llvm-commits.
delena added inline comments.Jan 14 2016, 12:09 PM
lib/Support/Host.cpp
826 ↗(On Diff #44887)

please take lower case avx512vbmi

lib/Target/X86/X86.td
135 ↗(On Diff #44887)

The string is copied from the prev case.

lib/Target/X86/X86InstrInfo.td
798 ↗(On Diff #44887)

remove !

m_zuckerman updated this object.Jan 17 2016, 1:56 AM
m_zuckerman updated this object.
m_zuckerman marked 2 inline comments as done.
delena accepted this revision.Jan 17 2016, 4:28 AM
delena edited edge metadata.

LGTM

This revision is now accepted and ready to land.Jan 17 2016, 4:28 AM
This revision was automatically updated to reflect the committed changes.