Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Rebased to work with top of the tree. There were issues with recently added RDHWR and RDPGPR instructions.
Comment Actions
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. |