This is an archive of the discontinued LLVM Phabricator instance.

[mips][microMIPS] Implement ERET and ERETNC instructions
ClosedPublic

Authored by zbuljan on May 28 2015, 8:30 AM.

Details

Summary

The patch implements microMIPSr6 ERET and ERETNC instructions.

Diff Detail

Event Timeline

zbuljan updated this revision to Diff 26689.May 28 2015, 8:30 AM
zbuljan retitled this revision from to [mips][microMIPS] Implement ERET and ERETNC instructions.
zbuljan updated this object.
zbuljan edited the test plan for this revision. (Show Details)
zbuljan added subscribers: petarj, Unknown Object (MLST).
dsanders accepted this revision.Jun 1 2015, 6:38 AM
dsanders edited edge metadata.

LGTM with a nit

lib/Target/Mips/MicroMips32r6InstrFormats.td
186–204

Nit: Each of these two formats only have one instruction using them. As such, we should remove the 'funct' argument and define the number here.

This revision is now accepted and ready to land.Jun 1 2015, 6:38 AM
dsanders added inline comments.Jun 1 2015, 6:42 AM
lib/Target/Mips/MicroMips32r6InstrFormats.td
186–204

One more nit. 'instr_asm' is unused and should be removed.

dsanders added inline comments.Jun 1 2015, 6:44 AM
lib/Target/Mips/MicroMips32r6InstrFormats.td
186–204

One more nit. 'instr_asm' is unused and should be removed.

Please ignore that. I was thinking of normal MIPS which doesn't have instruction mnemonics in the formats.

zbuljan closed this revision.Jun 18 2015, 3:53 AM

Closed by commit rL239522: [mips][microMIPS] Implement ERET and ERETNC instructions