This is an archive of the discontinued LLVM Phabricator instance.

AVX512 : vpextrb/w/d/q and vpinsrb/w/d/q implementation
ClosedPublic

Authored by igorb on Aug 25 2015, 4:47 AM.

Details

Summary

AVX512: vpextrb/w/d/q and vpinsrb/w/d/q implementation.
This instructions doesn't have intrincis.
Added tests for encoding.

Diff Detail

Repository
rL LLVM

Event Timeline

igorb updated this revision to Diff 33059.Aug 25 2015, 4:47 AM
igorb retitled this revision from to AVX512 : vpextrb/d/q and vpinsrb/w/d/q implementation.
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.Sep 7 2015, 11:25 PM
lib/Target/X86/X86InstrAVX512.td
6832

Please add tests for DAG lowering.

6845

DAG with "trunc" is not the main pattern. You should add DAG with GR16 and GR8 types.

igorb updated this revision to Diff 36594.Oct 6 2015, 1:56 AM
igorb retitled this revision from AVX512 : vpextrb/d/q and vpinsrb/w/d/q implementation to AVX512 : vpextrb/w/d/q and vpinsrb/w/d/q implementation.
igorb updated this object.
igorb marked 2 inline comments as done.
delena edited edge metadata.Oct 8 2015, 4:04 AM

LGTM

lib/Target/X86/X86InstrSSE.td
4614 ↗(On Diff #36594)

Please add tests for AVX

This revision was automatically updated to reflect the committed changes.