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. |