The patch implements microMIPS32r6 and microMIPS64r6 ABS.fmt, CEIL.L.fmt, CEIL.W.fmt, FLOOR.L.fmt, FLOOR.W.fmt, TRUNC.L.fmt, TRUNC.W.fmt, RSQRT.fmt and SQRT.fmt FPU instructions.
Details
Diff Detail
Event Timeline
lib/Target/Mips/MicroMips32r6InstrFormats.td | ||
---|---|---|
290–319 | As with the other patches, these need to follow the naming convention: POOL32F_<misc>_FM | |
lib/Target/Mips/MicroMips32r6InstrInfo.td | ||
71–89 | The double precision instructions should have a '_D' in the same way as the single precision ones have '_S' Also, there's some inconsistency in where the '_S' appears in these. Most have it before the 'MMR6' but a couple have it after. | |
90 | Nit: Extra line of whitespace | |
301–302 | Nit: space after comma and space either side of equals | |
430 | Nit: Indentation. Likewise below | |
lib/Target/Mips/MipsInstrFPU.td | ||
272 | I think you mean 'NotMicroMips32r6'. They are present in Mips32r6. | |
294 | I think you mean 'NotMicroMips32r6' | |
352 | I think you mean 'NotMicroMips32r6' | |
test/MC/Disassembler/Mips/micromips32r6.txt | ||
150 | Blank line at EOF |
As with the other patches, these need to follow the naming convention: