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 ↗(On Diff #33059)

Please add tests for DAG lowering.

6845 ↗(On Diff #33059)

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.