This patch implements microMIPS32r6 CFC1, CFC2, CTC1, CTC2, LDC1 and LDC2 instructions.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
Minor nits.
For the (in)valid tests, can you integrate them alphabetically into the corresponding instruction groups.
Can you rebase this to ToT?
| lib/Target/Mips/MicroMipsInstrFPU.td | ||
|---|---|---|
| 150 ↗ | (On Diff #52184) | Space between the FGR_32 and the {. |
| lib/Target/Mips/MipsInstrInfo.td | ||
| 1088 ↗ | (On Diff #52184) | Formatting, line length should not exceed 80 characters if possible. Start a newline after the new parameter (Operand MemOpnd). |
| 1647 ↗ | (On Diff #52184) | Formatting, line length should not exceed 80 characters. Start a new line after "load>,". |
| test/MC/Mips/mips32r6/invalid.s | ||
| 16 ↗ | (On Diff #52184) | What is being tested here that line 15 does not cover? |
| test/MC/Mips/mips64r6/invalid.s | ||
| 16 ↗ | (On Diff #52184) | See my comment on mips32r6/invalid.s. |
Updated according to comments received from sdardis and also rebased to work with TOT.
LGTM with the highlighted nits addressed.
| lib/Target/Mips/MicroMips32r6InstrInfo.td | ||
|---|---|---|
| 680–682 ↗ | (On Diff #54169) | Formatting should be like: class LDWC1_DESC_BASE<string opstr, RegisterOperand RC, InstrItinClass Itin,
SDPatternOperator OpNode = null_frag> : MipsR6Inst,
HARDFLOAT {You can start a newline in the parameter list. |
| 698–700 ↗ | (On Diff #54169) | The formatting should be like: class COP2LD_MMR6_DESC_BASE<string opstr, RegisterOperand COPOpnd,
InstrItinClass Itin,
SDPatternOperator OpNode = null_frag> { |
| test/MC/Mips/micromips32r6/valid.s | ||
| 42–45 ↗ | (On Diff #54169) | Line up the "# encoding: .." with the others. |
| 62–63 ↗ | (On Diff #54169) | And here. |
| test/MC/Mips/mips32r5/invalid.s | ||
| 14 ↗ | (On Diff #54169) | Line up this check line with the rest. The top one is ok. |
| test/MC/Mips/mips64r5/invalid.s | ||
| 20 ↗ | (On Diff #54169) | Same again here. |
| test/MC/Mips/mips64r6/invalid.s | ||
| 12 ↗ | (On Diff #54169) | Restore the jalr.hb lines and submit that as a separate NFC change. |
| 35–38 ↗ | (On Diff #54169) | Submit this as a separate change. |