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.