This is an archive of the discontinued LLVM Phabricator instance.

[mips][microMIPS] Implement PREFX, LHUE, LBE, LBUE, LHE, LWE, SBE, SHE and SWE instructions
ClosedPublic

Authored by zbuljan on Apr 22 2015, 5:06 AM.

Details

Summary

This patch implements microMIPS PREFX, LHUE, LBE, LBUE, LHE, LWE, SBE, SHE and SWE instructions.

Diff Detail

Repository
rL LLVM

Event Timeline

zbuljan updated this revision to Diff 24211.Apr 22 2015, 5:06 AM
zbuljan retitled this revision from to [mips][microMIPS] Implement PREFX, LHUE, LBE, LBUE, LHE, LWE, SBE, SHE and SWE instructions.
zbuljan updated this object.
zbuljan edited the test plan for this revision. (Show Details)
zbuljan added subscribers: petarj, Unknown Object (MLST).
zbuljan updated this revision to Diff 25299.May 8 2015, 3:06 AM

Added more context lines to diff.

dsanders accepted this revision.Sep 4 2015, 1:56 AM
dsanders edited edge metadata.

LGTM with a few nits

lib/Target/Mips/Disassembler/MipsDisassembler.cpp
281–283 ↗(On Diff #25299)

Indentation

1295–1297 ↗(On Diff #25299)

Indentation

lib/Target/Mips/MicroMipsInstrFormats.td
392 ↗(On Diff #25299)

This should start with POOL32C_

400–403 ↗(On Diff #25299)

Could you assign the RHS to fields (base and offset) first so that the special encoding is self documented?

939 ↗(On Diff #25299)

Should begin with POOL32F_

This revision is now accepted and ready to land.Sep 4 2015, 1:56 AM
This revision was automatically updated to reflect the committed changes.