This is an archive of the discontinued LLVM Phabricator instance.

AVX : Fix ISA disabling in case AVX512VL present
ClosedPublic

Authored by igorb on Jul 12 2015, 3:57 AM.

Details

Summary

AVX : Fix ISA disabling in case AVX512VL , some instructions should be disabled only if AVX512BW present.
Tests added.

Diff Detail

Repository
rL LLVM

Event Timeline

igorb updated this revision to Diff 29519.Jul 12 2015, 3:57 AM
igorb retitled this revision from to AVX : Fix ISA disabling in case AVX512VL present .
igorb updated this object.
igorb added reviewers: delena, AsafBadouh.
igorb set the repository for this revision to rL LLVM.
igorb added a subscriber: llvm-commits.
delena added inline comments.Jul 12 2015, 6:30 AM
lib/Target/X86/X86InstrSSE.td
4036

Please add NoBWI for all relevant templates (i8, i16) and explicit NoVLX for instructions with 32 and 64 bit elements.

test/CodeGen/X86/avx-isa-check.ll
2

Please add a comment what we are checking in this test.

igorb updated this revision to Diff 29655.Jul 14 2015, 2:40 AM
igorb marked 2 inline comments as done.

new diff uploaded.
all rejects fixed

This revision was automatically updated to reflect the committed changes.