Details
Diff Detail
Event Timeline
Rebased to work with top of the tree. There were issues with recently added RDHWR and RDPGPR instructions.
LGTM with a few nits.
The RDHWR/RDGPR/LUI changes should be in a separate patch
lib/Target/Mips/Disassembler/MipsDisassembler.cpp | ||
---|---|---|
2083–2168 | Please clang-format this | |
2101–2103 | Redundant braces | |
lib/Target/Mips/MicroMips32r6InstrFormats.td | ||
20–24 | Could you also add the 'Encoding Formats' one before the encoding formats begin? Also, MicroMipsR6Inst16 should probably be outside this section. | |
575 | 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. |
Redundant braces