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

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

Indentation

1295–1297

Indentation

lib/Target/Mips/MicroMipsInstrFormats.td
392

This should start with POOL32C_

400–403

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

939

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.