The patch implements microMIPSDSP instructions: SHLL.PH, SHLL_S.PH, SHLL.QB, SHLLV.PH, SHLLV_S.PH, SHLLV.QB, SHLLV_S.W, SHLL_S.W, SHRA.QB and SHRA_R.QB
Details
Diff Detail
Event Timeline
LGTM with a couple nits.
lib/Target/Mips/MicroMipsDSPInstrInfo.td | ||
---|---|---|
76 | Could you drop the '_QB'? This base class is used by the other SHLL's too. | |
80 | This uimm16 should be uimm3, uimm4, or uimm5 depending on the exact instruction. There's no functional difference right now but I'm starting to fix the missing range checks. If any of these don't exist then add them alongside the other uimm operand definitions. | |
101 | Could you clarify that this is SHLLV and not SHLL? I was confused by the lack of an immediate in operand list at first. Also, could you drop the '_QB'? This base class is used by the other SHLLV's too. |
Could you drop the '_QB'? This base class is used by the other SHLL's too.