[PATCH] Implement MUL, MUH, MULU and MUHU instructions

Daniel Sanders daniel.sanders at imgtec.com
Wed Apr 15 02:24:33 PDT 2015


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.


================
Comment at: lib/Target/Mips/MicroMips32r6InstrInfo.td:25
@@ +24,3 @@
+class MUL_MMR6_ENC : ARITH_FM_MMR6<"mul", 0x18>;
+class MUH_MMR6_ENC : ARITH_FM_MMR6<"muh", 0x2c>;
+class MULU_MMR6_ENC : ARITH_FM_MMR6<"mulu", 0x98>;
----------------
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.

================
Comment at: lib/Target/Mips/MicroMips32r6InstrInfo.td:27
@@ -24,1 +26,3 @@
+class MULU_MMR6_ENC : ARITH_FM_MMR6<"mulu", 0x98>;
+class MUHU_MMR6_ENC : ARITH_FM_MMR6<"muhu", 0xe8>;
 
----------------
I make it 0xd8.

http://reviews.llvm.org/D8894

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list