Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
LGTM except for a small issue in the muhu and possibly muh instructions. Once muhu is corrected and muh has been confirmed correct you can commit.
| lib/Target/Mips/MicroMips32r6InstrInfo.td | ||
|---|---|---|
| 25 | I believe this is correct, but the encoding diagram only has 9-bits specified for this 10-bit field. Presumably is a leading zero is missing. I've asked the relevant people to confirm and correct the documentation. | |
| 27 | I make it 0xd8. | |
| lib/Target/Mips/MicroMips32r6InstrInfo.td | ||
|---|---|---|
| 25 | I've had confirmation that it's actually a trailing zero that's missing so I believe this should be 0x58 | |
I believe this is correct, but the encoding diagram only has 9-bits specified for this 10-bit field. Presumably is a leading zero is missing. I've asked the relevant people to confirm and correct the documentation.