These instructions were only defined for microMIPSR6 previously. Add
definitions for MIPSR6, correct definitions for microMIPSR6, flag these
instructions as having unmodelled side effects (they disable/enable
virtual processors) and add missing disassember tests for microMIPSR6.
Details
- Reviewers
vkalintiris
Diff Detail
Event Timeline
LGTM with a few minor changes, see inline comments.
lib/Target/Mips/MicroMips32r6InstrInfo.td | ||
---|---|---|
1351 | I believe that this should be an out operand. | |
lib/Target/Mips/Mips32r6InstrFormats.td | ||
48 | I don't think that we need the _DVPEVP suffix because we might need the COP0 opcode group again in the future. | |
lib/Target/Mips/Mips32r6InstrInfo.td | ||
911–912 | Can you add tests for the aliases too? |
Thanks, committed as r284115.
lib/Target/Mips/MicroMips32r6InstrInfo.td | ||
---|---|---|
1351 | Yes, you're correct there. | |
lib/Target/Mips/Mips32r6InstrFormats.td | ||
48 | Rechecking the .td files shows that OPGROUP_COP0 was defined in MipsEVAInstrFormats.td. I've renamed the def there to OPGROUP_COP0_TLB. At some point we will want to refactor the format files. |
I believe that this should be an out operand.