This is an archive of the discontinued LLVM Phabricator instance.

[mips][microMIPS] Implement EXTP, EXTPDP, EXTPDPV, EXTPV, EXTR[_RS].W, EXTR_S.H, EXTRV[_RS].W and EXTRV_S.H instructions
ClosedPublic

Authored by zbuljan on Oct 29 2015, 6:36 AM.

Details

Summary

The patch implements microMIPSDSP instructions: EXTP, EXTPDP, EXTPDPV, EXTPV, EXTR[_RS].W, EXTR_S.H, EXTRV[_RS].W and EXTRV_S.H

Diff Detail

Repository
rL LLVM

Event Timeline

zbuljan updated this revision to Diff 38731.Oct 29 2015, 6:36 AM
zbuljan retitled this revision from to [mips][microMIPS] Implement EXTP, EXTPDP, EXTPDPV, EXTPV, EXTR[_RS].W, EXTR_S.H, EXTRV[_RS].W and EXTRV_S.H instructions.
zbuljan updated this object.
zbuljan added subscribers: petarj, llvm-commits.
dsanders accepted this revision.Nov 14 2015, 6:05 AM
dsanders edited edge metadata.

LGTM with a formatting nit.

I didn't double check all the implicit-uses and implicit-defs since I have a lot of reviews to catch up on but the first few looked good.

lib/Target/Mips/MicroMipsDSPInstrInfo.td
133–146 ↗(On Diff #38731)

Formatting nit: 'GPR32Opnd : $rt' -> 'GPR32Opnd:$rt' and similar.

This revision is now accepted and ready to land.Nov 14 2015, 6:05 AM

Sorry, I missed a small detail.

test/MC/Disassembler/Mips/micromips-dsp/valid.txt
31–42 ↗(On Diff #38731)

These should be copied to the DSPr2 tests too. Likewise for the assembler tests.

See my comment on D14058 for more info.

This revision was automatically updated to reflect the committed changes.