This is an archive of the discontinued LLVM Phabricator instance.

[mips][microMIPS] Implement BGEC, BGEUC, BLTC, BLTUC, BEQC and BNEC instructions
ClosedPublic

Authored by zoran.jovanovic on Oct 30 2015, 8:57 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

zoran.jovanovic retitled this revision from to [mips][microMIPS] Implement BGEC, BGEUC, BLTC, BLTUC, BEQC and BNEC instructions.
zoran.jovanovic updated this object.
zoran.jovanovic added a subscriber: llvm-commits.

Rebased to work with top of the tree. There were issues with recently added RDHWR and RDPGPR instructions.

dsanders accepted this revision.Dec 4 2015, 8:22 AM
dsanders edited edge metadata.

LGTM with a few nits.

The RDHWR/RDGPR/LUI changes should be in a separate patch

lib/Target/Mips/Disassembler/MipsDisassembler.cpp
2083–2168 ↗(On Diff #40053)

Please clang-format this

2101–2103 ↗(On Diff #40053)

Redundant braces

lib/Target/Mips/MicroMips32r6InstrFormats.td
20–24 ↗(On Diff #40053)

Could you also add the 'Encoding Formats' one before the encoding formats begin?

Also, MicroMipsR6Inst16 should probably be outside this section.

575 ↗(On Diff #40053)

The 'MMR6Arch<instr_asm>, MipsR6Inst' -> 'MipsR6Inst, MMR6Arch<instr_asm>' tidy up ought to be in a separate patch which can be committed without further review.

This revision is now accepted and ready to land.Dec 4 2015, 8:22 AM
This revision was automatically updated to reflect the committed changes.