Eliminate warning message.
[binutils-gdb.git] / opcodes / mips-dis.c
index 0d1de30f97ca7e6c3ce25bda32421370bb816214..018ac94f302c0c68e2ac7f3a339c18afcfb5857e 100644 (file)
@@ -1,6 +1,6 @@
 /* Print mips instructions for GDB, the GNU debugger, or for objdump.
    Copyright 1989, 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999,
-   2000, 2001, 2002, 2003, 2005, 2006, 2007, 2008, 2009
+   2000, 2001, 2002, 2003, 2005, 2006, 2007, 2008, 2009, 2012
    Free Software Foundation, Inc.
    Contributed by Nobuyuki Hikichi(hikichi@sra.co.jp).
 
@@ -233,6 +233,30 @@ static const char * const mips_cp0_names_r4000[32] =
   "c0_taglo",     "c0_taghi",     "c0_errorepc",  "$31",
 };
 
+static const char * const mips_cp0_names_r5900[32] =
+{
+  "c0_index",     "c0_random",    "c0_entrylo0",  "c0_entrylo1",
+  "c0_context",   "c0_pagemask",  "c0_wired",     "$7",
+  "c0_badvaddr",  "c0_count",     "c0_entryhi",   "c0_compare",
+  "c0_sr",        "c0_cause",     "c0_epc",       "c0_prid",
+  "c0_config",    "$17",          "$18",          "$19",
+  "$20",          "$21",          "$22",          "c0_badpaddr",
+  "c0_depc",      "c0_perfcnt",   "$26",          "$27",
+  "c0_taglo",     "c0_taghi",     "c0_errorepc",  "$31"
+};
+
+static const struct mips_cp0sel_name mips_cp0sel_names_mipsr5900[] =
+{
+  { 24, 2, "c0_iab"                    },
+  { 24, 3, "c0_iabm"           },
+  { 24, 4, "c0_dab"                    },
+  { 24, 5, "c0_dabm"           },
+  { 24, 6, "c0_dvb"                    },
+  { 24, 7, "c0_dvbm"           },
+  { 25, 1, "c0_perfcnt,1"      },
+  { 25, 2, "c0_perfcnt,2"      }
+};
+
 static const char * const mips_cp0_names_mips3264[32] =
 {
   "c0_index",     "c0_random",    "c0_entrylo0",  "c0_entrylo1",
@@ -532,6 +556,8 @@ const struct mips_arch_choice mips_arch_choices[] =
     mips_cp0_names_numeric, NULL, 0, mips_hwr_names_numeric },
   { "vr5500",  1, bfd_mach_mips5500, CPU_VR5500, ISA_MIPS4,
     mips_cp0_names_numeric, NULL, 0, mips_hwr_names_numeric },
+  { "r5900",   1, bfd_mach_mips5900, CPU_R5900, ISA_MIPS3,
+    mips_cp0_names_r5900, NULL, 0, mips_hwr_names_numeric },
   { "r6000",   1, bfd_mach_mips6000, CPU_R6000, ISA_MIPS2,
     mips_cp0_names_numeric, NULL, 0, mips_hwr_names_numeric },
   { "rm7000",  1, bfd_mach_mips7000, CPU_RM7000, ISA_MIPS4,
@@ -619,6 +645,14 @@ const struct mips_arch_choice mips_arch_choices[] =
     mips_cp0sel_names_xlr, ARRAY_SIZE (mips_cp0sel_names_xlr),
     mips_hwr_names_numeric },
 
+  /* XLP is mostly like XLR, with the prominent exception it is being
+     MIPS64R2.  */
+  { "xlp", 1, bfd_mach_mips_xlr, CPU_XLR,
+    ISA_MIPS64R2 | INSN_XLR,
+    mips_cp0_names_xlr,
+    mips_cp0sel_names_xlr, ARRAY_SIZE (mips_cp0sel_names_xlr),
+    mips_hwr_names_numeric },
+
   /* This entry, mips16, is here only for ISA/processor selection; do
      not print its name.  */
   { "",                1, bfd_mach_mips16, CPU_MIPS16, ISA_MIPS3,
@@ -922,16 +956,23 @@ lookup_mips_cp0sel_name (const struct mips_cp0sel_name *names,
 
 static void
 print_insn_args (const char *d,
-                register unsigned long int l,
+                int l,
                 bfd_vma pc,
                 struct disassemble_info *info,
                 const struct mips_opcode *opp)
 {
-  int op, delta;
+  const fprintf_ftype infprintf = info->fprintf_func;
   unsigned int lsb, msb, msbd;
+  void *is = info->stream;
+  int op;
 
   lsb = 0;
 
+#define GET_OP(insn, field) \
+  (((insn) >> OP_SH_##field) & OP_MASK_##field)
+#define GET_OP_S(insn, field) \
+  ((GET_OP (insn, field) ^ ((OP_MASK_##field >> 1) + 1)) \
+   - ((OP_MASK_##field >> 1) + 1))
   for (; *d != '\0'; d++)
     {
       switch (*d)
@@ -941,7 +982,7 @@ print_insn_args (const char *d,
        case ')':
        case '[':
        case ']':
-         (*info->fprintf_func) (info->stream, "%c", *d);
+         infprintf (is, "%c", *d);
          break;
 
        case '+':
@@ -951,44 +992,41 @@ print_insn_args (const char *d,
            {
            case '\0':
              /* xgettext:c-format */
-             (*info->fprintf_func) (info->stream,
-                                    _("# internal error, incomplete extension sequence (+)"));
+             infprintf (is,
+                        _("# internal error, "
+                          "incomplete extension sequence (+)"));
              return;
 
            case 'A':
-             lsb = (l >> OP_SH_SHAMT) & OP_MASK_SHAMT;
-             (*info->fprintf_func) (info->stream, "0x%x", lsb);
+             lsb = GET_OP (l, SHAMT);
+             infprintf (is, "0x%x", lsb);
              break;
        
            case 'B':
-             msb = (l >> OP_SH_INSMSB) & OP_MASK_INSMSB;
-             (*info->fprintf_func) (info->stream, "0x%x", msb - lsb + 1);
+             msb = GET_OP (l, INSMSB);
+             infprintf (is, "0x%x", msb - lsb + 1);
              break;
 
            case '1':
-             (*info->fprintf_func) (info->stream, "0x%lx",
-                                    (l >> OP_SH_UDI1) & OP_MASK_UDI1);
+             infprintf (is, "0x%x", GET_OP (l, UDI1));
              break;
              
            case '2':
-             (*info->fprintf_func) (info->stream, "0x%lx",
-                                    (l >> OP_SH_UDI2) & OP_MASK_UDI2);
+             infprintf (is, "0x%x", GET_OP (l, UDI2));
              break;
              
            case '3':
-             (*info->fprintf_func) (info->stream, "0x%lx",
-                                    (l >> OP_SH_UDI3) & OP_MASK_UDI3);
+             infprintf (is, "0x%x", GET_OP (l, UDI3));
              break;
       
            case '4':
-             (*info->fprintf_func) (info->stream, "0x%lx",
-                                    (l >> OP_SH_UDI4) & OP_MASK_UDI4);
+             infprintf (is, "0x%x", GET_OP (l, UDI4));
              break;
              
            case 'C':
            case 'H':
-             msbd = (l >> OP_SH_EXTMSBD) & OP_MASK_EXTMSBD;
-             (*info->fprintf_func) (info->stream, "0x%x", msbd + 1);
+             msbd = GET_OP (l, EXTMSBD);
+             infprintf (is, "0x%x", msbd + 1);
              break;
 
            case 'D':
@@ -996,8 +1034,8 @@ print_insn_args (const char *d,
                const struct mips_cp0sel_name *n;
                unsigned int cp0reg, sel;
 
-               cp0reg = (l >> OP_SH_RD) & OP_MASK_RD;
-               sel = (l >> OP_SH_SEL) & OP_MASK_SEL;
+               cp0reg = GET_OP (l, RD);
+               sel = GET_OP (l, SEL);
 
                /* CP0 register including 'sel' code for mtcN (et al.), to be
                   printed textually if known.  If not known, print both
@@ -1007,31 +1045,29 @@ print_insn_args (const char *d,
                n = lookup_mips_cp0sel_name(mips_cp0sel_names,
                                            mips_cp0sel_names_len, cp0reg, sel);
                if (n != NULL)
-                 (*info->fprintf_func) (info->stream, "%s", n->name);
+                 infprintf (is, "%s", n->name);
                else
-                 (*info->fprintf_func) (info->stream, "$%d,%d", cp0reg, sel);
+                 infprintf (is, "$%d,%d", cp0reg, sel);
                break;
              }
 
            case 'E':
-             lsb = ((l >> OP_SH_SHAMT) & OP_MASK_SHAMT) + 32;
-             (*info->fprintf_func) (info->stream, "0x%x", lsb);
+             lsb = GET_OP (l, SHAMT) + 32;
+             infprintf (is, "0x%x", lsb);
              break;
        
            case 'F':
-             msb = ((l >> OP_SH_INSMSB) & OP_MASK_INSMSB) + 32;
-             (*info->fprintf_func) (info->stream, "0x%x", msb - lsb + 1);
+             msb = GET_OP (l, INSMSB) + 32;
+             infprintf (is, "0x%x", msb - lsb + 1);
              break;
 
            case 'G':
-             msbd = ((l >> OP_SH_EXTMSBD) & OP_MASK_EXTMSBD) + 32;
-             (*info->fprintf_func) (info->stream, "0x%x", msbd + 1);
+             msbd = GET_OP (l, EXTMSBD) + 32;
+             infprintf (is, "0x%x", msbd + 1);
              break;
 
            case 't': /* Coprocessor 0 reg name */
-             (*info->fprintf_func) (info->stream, "%s",
-                                    mips_cp0_names[(l >> OP_SH_RT) &
-                                                    OP_MASK_RT]);
+             infprintf (is, "%s", mips_cp0_names[GET_OP (l, RT)]);
              break;
 
            case 'T': /* Coprocessor 0 reg name */
@@ -1039,8 +1075,8 @@ print_insn_args (const char *d,
                const struct mips_cp0sel_name *n;
                unsigned int cp0reg, sel;
 
-               cp0reg = (l >> OP_SH_RT) & OP_MASK_RT;
-               sel = (l >> OP_SH_SEL) & OP_MASK_SEL;
+               cp0reg = GET_OP (l, RT);
+               sel = GET_OP (l, SEL);
 
                /* CP0 register including 'sel' code for mftc0, to be
                   printed textually if known.  If not known, print both
@@ -1050,228 +1086,173 @@ print_insn_args (const char *d,
                n = lookup_mips_cp0sel_name(mips_cp0sel_names,
                                            mips_cp0sel_names_len, cp0reg, sel);
                if (n != NULL)
-                 (*info->fprintf_func) (info->stream, "%s", n->name);
+                 infprintf (is, "%s", n->name);
                else
-                 (*info->fprintf_func) (info->stream, "$%d,%d", cp0reg, sel);
+                 infprintf (is, "$%d,%d", cp0reg, sel);
                break;
              }
 
            case 'x':           /* bbit bit index */
-             (*info->fprintf_func) (info->stream, "0x%lx",
-                                    (l >> OP_SH_BBITIND) & OP_MASK_BBITIND);
+             infprintf (is, "0x%x", GET_OP (l, BBITIND));
              break;
 
            case 'p':           /* cins, cins32, exts and exts32 position */
-             (*info->fprintf_func) (info->stream, "0x%lx",
-                                    (l >> OP_SH_CINSPOS) & OP_MASK_CINSPOS);
+             infprintf (is, "0x%x", GET_OP (l, CINSPOS));
              break;
 
            case 's':           /* cins and exts length-minus-one */
-             (*info->fprintf_func) (info->stream, "0x%lx",
-                                    (l >> OP_SH_CINSLM1) & OP_MASK_CINSLM1);
+             infprintf (is, "0x%x", GET_OP (l, CINSLM1));
              break;
 
            case 'S':           /* cins32 and exts32 length-minus-one field */
-             (*info->fprintf_func) (info->stream, "0x%lx",
-                                    (l >> OP_SH_CINSLM1) & OP_MASK_CINSLM1);
+             infprintf (is, "0x%x", GET_OP (l, CINSLM1));
              break;
 
            case 'Q':           /* seqi/snei immediate field */
-             op = (l >> OP_SH_SEQI) & OP_MASK_SEQI;
-             /* Sign-extend it.  */
-             op = (op ^ 512) - 512;
-             (*info->fprintf_func) (info->stream, "%d", op);
+             infprintf (is, "%d", GET_OP_S (l, SEQI));
              break;
 
            case 'a':           /* 8-bit signed offset in bit 6 */
-             delta = (l >> OP_SH_OFFSET_A) & OP_MASK_OFFSET_A;
-             if (delta & 0x80)
-               delta |= ~OP_MASK_OFFSET_A;
-             (*info->fprintf_func) (info->stream, "%d", delta);
+             infprintf (is, "%d", GET_OP_S (l, OFFSET_A));
              break;
 
            case 'b':           /* 8-bit signed offset in bit 3 */
-             delta = (l >> OP_SH_OFFSET_B) & OP_MASK_OFFSET_B;
-             if (delta & 0x80)
-               delta |= ~OP_MASK_OFFSET_B;
-             (*info->fprintf_func) (info->stream, "%d", delta);
+             infprintf (is, "%d", GET_OP_S (l, OFFSET_B));
              break;
 
            case 'c':           /* 9-bit signed offset in bit 6 */
-             delta = (l >> OP_SH_OFFSET_C) & OP_MASK_OFFSET_C;
-             if (delta & 0x100)
-               delta |= ~OP_MASK_OFFSET_C;
              /* Left shift 4 bits to print the real offset.  */
-             (*info->fprintf_func) (info->stream, "%d", delta << 4);
+             infprintf (is, "%d", GET_OP_S (l, OFFSET_C) << 4);
              break;
 
            case 'z':
-             (*info->fprintf_func) (info->stream, "%s",
-                                    mips_gpr_names[(l >> OP_SH_RZ) & OP_MASK_RZ]);
+             infprintf (is, "%s", mips_gpr_names[GET_OP (l, RZ)]);
              break;
 
            case 'Z':
-             (*info->fprintf_func) (info->stream, "%s",
-                                    mips_fpr_names[(l >> OP_SH_FZ) & OP_MASK_FZ]);
+             infprintf (is, "%s", mips_fpr_names[GET_OP (l, FZ)]);
              break;
 
            default:
              /* xgettext:c-format */
-             (*info->fprintf_func) (info->stream,
-                                    _("# internal error, undefined extension sequence (+%c)"),
-                                    *d);
+             infprintf (is,
+                        _("# internal error, "
+                          "undefined extension sequence (+%c)"),
+                        *d);
              return;
            }
          break;
 
        case '2':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_BP) & OP_MASK_BP);
+         infprintf (is, "0x%x", GET_OP (l, BP));
          break;
 
        case '3':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_SA3) & OP_MASK_SA3);
+         infprintf (is, "0x%x", GET_OP (l, SA3));
          break;
 
        case '4':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_SA4) & OP_MASK_SA4);
+         infprintf (is, "0x%x", GET_OP (l, SA4));
          break;
 
        case '5':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_IMM8) & OP_MASK_IMM8);
+         infprintf (is, "0x%x", GET_OP (l, IMM8));
          break;
 
        case '6':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_RS) & OP_MASK_RS);
+         infprintf (is, "0x%x", GET_OP (l, RS));
          break;
 
        case '7':
-         (*info->fprintf_func) (info->stream, "$ac%ld",
-                                (l >> OP_SH_DSPACC) & OP_MASK_DSPACC);
+         infprintf (is, "$ac%d", GET_OP (l, DSPACC));
          break;
 
        case '8':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_WRDSP) & OP_MASK_WRDSP);
+         infprintf (is, "0x%x", GET_OP (l, WRDSP));
          break;
 
        case '9':
-         (*info->fprintf_func) (info->stream, "$ac%ld",
-                                (l >> OP_SH_DSPACC_S) & OP_MASK_DSPACC_S);
+         infprintf (is, "$ac%d", GET_OP (l, DSPACC_S));
          break;
 
        case '0': /* dsp 6-bit signed immediate in bit 20 */
-         delta = ((l >> OP_SH_DSPSFT) & OP_MASK_DSPSFT);
-         if (delta & 0x20) /* test sign bit */
-           delta |= ~OP_MASK_DSPSFT;
-         (*info->fprintf_func) (info->stream, "%d", delta);
+         infprintf (is, "%d", GET_OP_S (l, DSPSFT));
          break;
 
        case ':': /* dsp 7-bit signed immediate in bit 19 */
-         delta = ((l >> OP_SH_DSPSFT_7) & OP_MASK_DSPSFT_7);
-         if (delta & 0x40) /* test sign bit */
-           delta |= ~OP_MASK_DSPSFT_7;
-         (*info->fprintf_func) (info->stream, "%d", delta);
+         infprintf (is, "%d", GET_OP_S (l, DSPSFT_7));
          break;
 
        case '~':
-         delta = (l >> OP_SH_OFFSET12) & OP_MASK_OFFSET12;
-         if (delta & 0x800)
-           delta |= ~0x7ff;
-         (*info->fprintf_func) (info->stream, "%d", delta);
+         infprintf (is, "%d", GET_OP_S (l, OFFSET12));
          break;
 
        case '\\':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_3BITPOS) & OP_MASK_3BITPOS);
+         infprintf (is, "0x%x", GET_OP (l, 3BITPOS));
          break;
 
        case '\'':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_RDDSP) & OP_MASK_RDDSP);
+         infprintf (is, "0x%x", GET_OP (l, RDDSP));
          break;
 
        case '@': /* dsp 10-bit signed immediate in bit 16 */
-         delta = ((l >> OP_SH_IMM10) & OP_MASK_IMM10);
-         if (delta & 0x200) /* test sign bit */
-           delta |= ~OP_MASK_IMM10;
-         (*info->fprintf_func) (info->stream, "%d", delta);
+         infprintf (is, "%d", GET_OP_S (l, IMM10));
          break;
 
        case '!':
-         (*info->fprintf_func) (info->stream, "%ld",
-                                (l >> OP_SH_MT_U) & OP_MASK_MT_U);
+         infprintf (is, "%d", GET_OP (l, MT_U));
          break;
 
        case '$':
-         (*info->fprintf_func) (info->stream, "%ld",
-                                (l >> OP_SH_MT_H) & OP_MASK_MT_H);
+         infprintf (is, "%d", GET_OP (l, MT_H));
          break;
 
        case '*':
-         (*info->fprintf_func) (info->stream, "$ac%ld",
-                                (l >> OP_SH_MTACC_T) & OP_MASK_MTACC_T);
+         infprintf (is, "$ac%d", GET_OP (l, MTACC_T));
          break;
 
        case '&':
-         (*info->fprintf_func) (info->stream, "$ac%ld",
-                                (l >> OP_SH_MTACC_D) & OP_MASK_MTACC_D);
+         infprintf (is, "$ac%d", GET_OP (l, MTACC_D));
          break;
 
        case 'g':
          /* Coprocessor register for CTTC1, MTTC2, MTHC2, CTTC2.  */
-         (*info->fprintf_func) (info->stream, "$%ld",
-                                (l >> OP_SH_RD) & OP_MASK_RD);
+         infprintf (is, "$%d", GET_OP (l, RD));
          break;
 
        case 's':
        case 'b':
        case 'r':
        case 'v':
-         (*info->fprintf_func) (info->stream, "%s",
-                                mips_gpr_names[(l >> OP_SH_RS) & OP_MASK_RS]);
+         infprintf (is, "%s", mips_gpr_names[GET_OP (l, RS)]);
          break;
 
        case 't':
        case 'w':
-         (*info->fprintf_func) (info->stream, "%s",
-                                mips_gpr_names[(l >> OP_SH_RT) & OP_MASK_RT]);
+         infprintf (is, "%s", mips_gpr_names[GET_OP (l, RT)]);
          break;
 
        case 'i':
        case 'u':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_IMMEDIATE) & OP_MASK_IMMEDIATE);
+         infprintf (is, "0x%x", GET_OP (l, IMMEDIATE));
          break;
 
        case 'j': /* Same as i, but sign-extended.  */
        case 'o':
-         delta = (l >> OP_SH_DELTA) & OP_MASK_DELTA;
-         if (delta & 0x8000)
-           delta |= ~0xffff;
-         (*info->fprintf_func) (info->stream, "%d",
-                                delta);
+         infprintf (is, "%d", GET_OP_S (l, DELTA));
          break;
 
        case 'h':
-         (*info->fprintf_func) (info->stream, "0x%x",
-                                (unsigned int) ((l >> OP_SH_PREFX)
-                                                & OP_MASK_PREFX));
+         infprintf (is, "0x%x", GET_OP (l, PREFX));
          break;
 
        case 'k':
-         (*info->fprintf_func) (info->stream, "0x%x",
-                                (unsigned int) ((l >> OP_SH_CACHE)
-                                                & OP_MASK_CACHE));
+         infprintf (is, "0x%x", GET_OP (l, CACHE));
          break;
 
        case 'a':
          info->target = (((pc + 4) & ~(bfd_vma) 0x0fffffff)
-                         | (((l >> OP_SH_TARGET) & OP_MASK_TARGET) << 2));
+                         | (GET_OP (l, TARGET) << 2));
          /* For gdb disassembler, force odd address on jalx.  */
          if (info->flavour == bfd_target_unknown_flavour
              && strcmp (opp->name, "jalx") == 0)
@@ -1281,98 +1262,82 @@ print_insn_args (const char *d,
 
        case 'p':
          /* Sign extend the displacement.  */
-         delta = (l >> OP_SH_DELTA) & OP_MASK_DELTA;
-         if (delta & 0x8000)
-           delta |= ~0xffff;
-         info->target = (delta << 2) + pc + INSNLEN;
+         info->target = (GET_OP_S (l, DELTA) << 2) + pc + INSNLEN;
          (*info->print_address_func) (info->target, info);
          break;
 
        case 'd':
-         (*info->fprintf_func) (info->stream, "%s",
-                                mips_gpr_names[(l >> OP_SH_RD) & OP_MASK_RD]);
+         infprintf (is, "%s", mips_gpr_names[GET_OP (l, RD)]);
          break;
 
        case 'U':
          {
            /* First check for both rd and rt being equal.  */
-           unsigned int reg = (l >> OP_SH_RD) & OP_MASK_RD;
-           if (reg == ((l >> OP_SH_RT) & OP_MASK_RT))
-             (*info->fprintf_func) (info->stream, "%s",
-                                    mips_gpr_names[reg]);
+           unsigned int reg;
+
+           reg = GET_OP (l, RD);
+           if (reg == GET_OP (l, RT))
+             infprintf (is, "%s", mips_gpr_names[reg]);
            else
              {
                /* If one is zero use the other.  */
                if (reg == 0)
-                 (*info->fprintf_func) (info->stream, "%s",
-                                        mips_gpr_names[(l >> OP_SH_RT) & OP_MASK_RT]);
-               else if (((l >> OP_SH_RT) & OP_MASK_RT) == 0)
-                 (*info->fprintf_func) (info->stream, "%s",
-                                        mips_gpr_names[reg]);
+                 infprintf (is, "%s", mips_gpr_names[GET_OP (l, RT)]);
+               else if (GET_OP (l, RT) == 0)
+                 infprintf (is, "%s", mips_gpr_names[reg]);
                else /* Bogus, result depends on processor.  */
-                 (*info->fprintf_func) (info->stream, "%s or %s",
-                                        mips_gpr_names[reg],
-                                        mips_gpr_names[(l >> OP_SH_RT) & OP_MASK_RT]);
+                 infprintf (is, "%s or %s",
+                            mips_gpr_names[reg],
+                            mips_gpr_names[GET_OP (l, RT)]);
              }
          }
          break;
 
        case 'z':
-         (*info->fprintf_func) (info->stream, "%s", mips_gpr_names[0]);
+         infprintf (is, "%s", mips_gpr_names[0]);
          break;
 
        case '<':
        case '1':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_SHAMT) & OP_MASK_SHAMT);
+         infprintf (is, "0x%x", GET_OP (l, SHAMT));
          break;
 
        case 'c':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_CODE) & OP_MASK_CODE);
+         infprintf (is, "0x%x", GET_OP (l, CODE));
          break;
 
        case 'q':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_CODE2) & OP_MASK_CODE2);
+         infprintf (is, "0x%x", GET_OP (l, CODE2));
          break;
 
        case 'C':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_COPZ) & OP_MASK_COPZ);
+         infprintf (is, "0x%x", GET_OP (l, COPZ));
          break;
 
        case 'B':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-
-                                (l >> OP_SH_CODE20) & OP_MASK_CODE20);
+         infprintf (is, "0x%x", GET_OP (l, CODE20));
          break;
 
        case 'J':
-         (*info->fprintf_func) (info->stream, "0x%lx",
-                                (l >> OP_SH_CODE19) & OP_MASK_CODE19);
+         infprintf (is, "0x%x", GET_OP (l, CODE19));
          break;
 
        case 'S':
        case 'V':
-         (*info->fprintf_func) (info->stream, "%s",
-                                mips_fpr_names[(l >> OP_SH_FS) & OP_MASK_FS]);
+         infprintf (is, "%s", mips_fpr_names[GET_OP (l, FS)]);
          break;
 
        case 'T':
        case 'W':
-         (*info->fprintf_func) (info->stream, "%s",
-                                mips_fpr_names[(l >> OP_SH_FT) & OP_MASK_FT]);
+         infprintf (is, "%s", mips_fpr_names[GET_OP (l, FT)]);
          break;
 
        case 'D':
-         (*info->fprintf_func) (info->stream, "%s",
-                                mips_fpr_names[(l >> OP_SH_FD) & OP_MASK_FD]);
+         infprintf (is, "%s", mips_fpr_names[GET_OP (l, FD)]);
          break;
 
        case 'R':
-         (*info->fprintf_func) (info->stream, "%s",
-                                mips_fpr_names[(l >> OP_SH_FR) & OP_MASK_FR]);
+         infprintf (is, "%s", mips_fpr_names[GET_OP (l, FR)]);
          break;
 
        case 'E':
@@ -1383,8 +1348,7 @@ print_insn_args (const char *d,
             'T' format.  Therefore, until we gain understanding of
             cp2 register names, we can simply print the register
             numbers.  */
-         (*info->fprintf_func) (info->stream, "$%ld",
-                                (l >> OP_SH_RT) & OP_MASK_RT);
+         infprintf (is, "$%d", GET_OP (l, RT));
          break;
 
        case 'G':
@@ -1392,60 +1356,50 @@ print_insn_args (const char *d,
             that FPU (cp1) instructions disassemble this field using
             'S' format.  Therefore, we only need to worry about cp0,
             cp2, and cp3.  */
-         op = (l >> OP_SH_OP) & OP_MASK_OP;
+         op = GET_OP (l, OP);
          if (op == OP_OP_COP0)
-           (*info->fprintf_func) (info->stream, "%s",
-                                  mips_cp0_names[(l >> OP_SH_RD) & OP_MASK_RD]);
+           infprintf (is, "%s", mips_cp0_names[GET_OP (l, RD)]);
          else
-           (*info->fprintf_func) (info->stream, "$%ld",
-                                  (l >> OP_SH_RD) & OP_MASK_RD);
+           infprintf (is, "$%d", GET_OP (l, RD));
          break;
 
        case 'K':
-         (*info->fprintf_func) (info->stream, "%s",
-                                mips_hwr_names[(l >> OP_SH_RD) & OP_MASK_RD]);
+         infprintf (is, "%s", mips_hwr_names[GET_OP (l, RD)]);
          break;
 
        case 'N':
-         (*info->fprintf_func) (info->stream,
-                                ((opp->pinfo & (FP_D | FP_S)) != 0
-                                 ? "$fcc%ld" : "$cc%ld"),
-                                (l >> OP_SH_BCC) & OP_MASK_BCC);
+         infprintf (is,
+                    (opp->pinfo & (FP_D | FP_S)) != 0 ? "$fcc%d" : "$cc%d",
+                    GET_OP (l, BCC));
          break;
 
        case 'M':
-         (*info->fprintf_func) (info->stream, "$fcc%ld",
-                                (l >> OP_SH_CCC) & OP_MASK_CCC);
+         infprintf (is, "$fcc%d", GET_OP (l, CCC));
          break;
 
        case 'P':
-         (*info->fprintf_func) (info->stream, "%ld",
-                                (l >> OP_SH_PERFREG) & OP_MASK_PERFREG);
+         infprintf (is, "%d", GET_OP (l, PERFREG));
          break;
 
        case 'e':
-         (*info->fprintf_func) (info->stream, "%ld",
-                                (l >> OP_SH_VECBYTE) & OP_MASK_VECBYTE);
+         infprintf (is, "%d", GET_OP (l, VECBYTE));
          break;
 
        case '%':
-         (*info->fprintf_func) (info->stream, "%ld",
-                                (l >> OP_SH_VECALIGN) & OP_MASK_VECALIGN);
+         infprintf (is, "%d", GET_OP (l, VECALIGN));
          break;
 
        case 'H':
-         (*info->fprintf_func) (info->stream, "%ld",
-                                (l >> OP_SH_SEL) & OP_MASK_SEL);
+         infprintf (is, "%d", GET_OP (l, SEL));
          break;
 
        case 'O':
-         (*info->fprintf_func) (info->stream, "%ld",
-                                (l >> OP_SH_ALN) & OP_MASK_ALN);
+         infprintf (is, "%d", GET_OP (l, ALN));
          break;
 
        case 'Q':
          {
-           unsigned int vsel = (l >> OP_SH_VSEL) & OP_MASK_VSEL;
+           unsigned int vsel = GET_OP (l, VSEL);
 
            if ((vsel & 0x10) == 0)
              {
@@ -1455,43 +1409,34 @@ print_insn_args (const char *d,
                for (fmt = 0; fmt < 3; fmt++, vsel >>= 1)
                  if ((vsel & 1) == 0)
                    break;
-               (*info->fprintf_func) (info->stream, "$v%ld[%d]",
-                                      (l >> OP_SH_FT) & OP_MASK_FT,
-                                      vsel >> 1);
+               infprintf (is, "$v%d[%d]", GET_OP (l, FT), vsel >> 1);
              }
            else if ((vsel & 0x08) == 0)
              {
-               (*info->fprintf_func) (info->stream, "$v%ld",
-                                      (l >> OP_SH_FT) & OP_MASK_FT);
+               infprintf (is, "$v%d", GET_OP (l, FT));
              }
            else
              {
-               (*info->fprintf_func) (info->stream, "0x%lx",
-                                      (l >> OP_SH_FT) & OP_MASK_FT);
+               infprintf (is, "0x%x", GET_OP (l, FT));
              }
          }
          break;
 
        case 'X':
-         (*info->fprintf_func) (info->stream, "$v%ld",
-                                (l >> OP_SH_FD) & OP_MASK_FD);
+         infprintf (is, "$v%d", GET_OP (l, FD));
          break;
 
        case 'Y':
-         (*info->fprintf_func) (info->stream, "$v%ld",
-                                (l >> OP_SH_FS) & OP_MASK_FS);
+         infprintf (is, "$v%d", GET_OP (l, FS));
          break;
 
        case 'Z':
-         (*info->fprintf_func) (info->stream, "$v%ld",
-                                (l >> OP_SH_FT) & OP_MASK_FT);
+         infprintf (is, "$v%d", GET_OP (l, FT));
          break;
 
        default:
          /* xgettext:c-format */
-         (*info->fprintf_func) (info->stream,
-                                _("# internal error, undefined modifier (%c)"),
-                                *d);
+         infprintf (is, _("# internal error, undefined modifier (%c)"), *d);
          return;
        }
     }
@@ -1504,12 +1449,14 @@ print_insn_args (const char *d,
 
 static int
 print_insn_mips (bfd_vma memaddr,
-                unsigned long int word,
+                int word,
                 struct disassemble_info *info)
 {
+  static const struct mips_opcode *mips_hash[OP_MASK_OP + 1];
+  const fprintf_ftype infprintf = info->fprintf_func;
   const struct mips_opcode *op;
   static bfd_boolean init = 0;
-  static const struct mips_opcode *mips_hash[OP_MASK_OP + 1];
+  void *is = info->stream;
 
   /* Build a hash table to shorten the search time.  */
   if (! init)
@@ -1523,7 +1470,7 @@ print_insn_mips (bfd_vma memaddr,
              if (op->pinfo == INSN_MACRO
                  || (no_aliases && (op->pinfo2 & INSN2_ALIAS)))
                continue;
-             if (i == ((op->match >> OP_SH_OP) & OP_MASK_OP))
+             if (i == GET_OP (op->match, OP))
                {
                  mips_hash[i] = op;
                  break;
@@ -1543,7 +1490,7 @@ print_insn_mips (bfd_vma memaddr,
   info->target = 0;
   info->target2 = 0;
 
-  op = mips_hash[(word >> OP_SH_OP) & OP_MASK_OP];
+  op = mips_hash[GET_OP (word, OP)];
   if (op != NULL)
     {
       for (; op < &mips_opcodes[NUMOPCODES]; op++)
@@ -1555,7 +1502,7 @@ print_insn_mips (bfd_vma memaddr,
              const char *d;
 
              /* We always allow to disassemble the jalx instruction.  */
-             if (! OPCODE_IS_MEMBER (op, mips_isa, mips_processor)
+             if (!opcode_is_member (op, mips_isa, mips_processor)
                  && strcmp (op->name, "jalx"))
                continue;
 
@@ -1582,12 +1529,12 @@ print_insn_mips (bfd_vma memaddr,
                                     | INSN_LOAD_MEMORY_DELAY)) != 0)
                info->insn_type = dis_dref;
 
-             (*info->fprintf_func) (info->stream, "%s", op->name);
+             infprintf (is, "%s", op->name);
 
              d = op->args;
              if (d != NULL && *d != '\0')
                {
-                 (*info->fprintf_func) (info->stream, "\t");
+                 infprintf (is, "\t");
                  print_insn_args (d, word, memaddr, info, op);
                }
 
@@ -1595,10 +1542,12 @@ print_insn_mips (bfd_vma memaddr,
            }
        }
     }
+#undef GET_OP_S
+#undef GET_OP
 
   /* Handle undefined instructions.  */
   info->insn_type = dis_noninsn;
-  (*info->fprintf_func) (info->stream, "0x%lx", word);
+  infprintf (is, "0x%x", word);
   return INSNLEN;
 }
 \f
@@ -1613,65 +1562,62 @@ print_mips16_insn_arg (char type,
                       bfd_vma memaddr,
                       struct disassemble_info *info)
 {
+  const fprintf_ftype infprintf = info->fprintf_func;
+  void *is = info->stream;
+
+#define GET_OP(insn, field) \
+  (((insn) >> MIPS16OP_SH_##field) & MIPS16OP_MASK_##field)
+#define GET_OP_S(insn, field) \
+  ((GET_OP (insn, field) ^ ((MIPS16OP_MASK_##field >> 1) + 1)) \
+   - ((MIPS16OP_MASK_##field >> 1) + 1))
   switch (type)
     {
     case ',':
     case '(':
     case ')':
-      (*info->fprintf_func) (info->stream, "%c", type);
+      infprintf (is, "%c", type);
       break;
 
     case 'y':
     case 'w':
-      (*info->fprintf_func) (info->stream, "%s",
-                            mips16_reg_names(((l >> MIPS16OP_SH_RY)
-                                              & MIPS16OP_MASK_RY)));
+      infprintf (is, "%s", mips16_reg_names (GET_OP (l, RY)));
       break;
 
     case 'x':
     case 'v':
-      (*info->fprintf_func) (info->stream, "%s",
-                            mips16_reg_names(((l >> MIPS16OP_SH_RX)
-                                              & MIPS16OP_MASK_RX)));
+      infprintf (is, "%s", mips16_reg_names (GET_OP (l, RX)));
       break;
 
     case 'z':
-      (*info->fprintf_func) (info->stream, "%s",
-                            mips16_reg_names(((l >> MIPS16OP_SH_RZ)
-                                              & MIPS16OP_MASK_RZ)));
+      infprintf (is, "%s", mips16_reg_names (GET_OP (l, RZ)));
       break;
 
     case 'Z':
-      (*info->fprintf_func) (info->stream, "%s",
-                            mips16_reg_names(((l >> MIPS16OP_SH_MOVE32Z)
-                                              & MIPS16OP_MASK_MOVE32Z)));
+      infprintf (is, "%s", mips16_reg_names (GET_OP (l, MOVE32Z)));
       break;
 
     case '0':
-      (*info->fprintf_func) (info->stream, "%s", mips_gpr_names[0]);
+      infprintf (is, "%s", mips_gpr_names[0]);
       break;
 
     case 'S':
-      (*info->fprintf_func) (info->stream, "%s", mips_gpr_names[29]);
+      infprintf (is, "%s", mips_gpr_names[29]);
       break;
 
     case 'P':
-      (*info->fprintf_func) (info->stream, "$pc");
+      infprintf (is, "$pc");
       break;
 
     case 'R':
-      (*info->fprintf_func) (info->stream, "%s", mips_gpr_names[31]);
+      infprintf (is, "%s", mips_gpr_names[31]);
       break;
 
     case 'X':
-      (*info->fprintf_func) (info->stream, "%s",
-                            mips_gpr_names[((l >> MIPS16OP_SH_REGR32)
-                                           & MIPS16OP_MASK_REGR32)]);
+      infprintf (is, "%s", mips_gpr_names[GET_OP (l, REGR32)]);
       break;
 
     case 'Y':
-      (*info->fprintf_func) (info->stream, "%s",
-                            mips_gpr_names[MIPS16OP_EXTRACT_REG32R (l)]);
+      infprintf (is, "%s", mips_gpr_names[MIPS16OP_EXTRACT_REG32R (l)]);
       break;
 
     case '<':
@@ -1709,51 +1655,51 @@ print_mips16_insn_arg (char type,
          {
          case '<':
            nbits = 3;
-           immed = (l >> MIPS16OP_SH_RZ) & MIPS16OP_MASK_RZ;
+           immed = GET_OP (l, RZ);
            extbits = 5;
            extu = 1;
            break;
          case '>':
            nbits = 3;
-           immed = (l >> MIPS16OP_SH_RX) & MIPS16OP_MASK_RX;
+           immed = GET_OP (l, RX);
            extbits = 5;
            extu = 1;
            break;
          case '[':
            nbits = 3;
-           immed = (l >> MIPS16OP_SH_RZ) & MIPS16OP_MASK_RZ;
+           immed = GET_OP (l, RZ);
            extbits = 6;
            extu = 1;
            break;
          case ']':
            nbits = 3;
-           immed = (l >> MIPS16OP_SH_RX) & MIPS16OP_MASK_RX;
+           immed = GET_OP (l, RX);
            extbits = 6;
            extu = 1;
            break;
          case '4':
            nbits = 4;
-           immed = (l >> MIPS16OP_SH_IMM4) & MIPS16OP_MASK_IMM4;
+           immed = GET_OP (l, IMM4);
            signedp = 1;
            extbits = 15;
            break;
          case '5':
            nbits = 5;
-           immed = (l >> MIPS16OP_SH_IMM5) & MIPS16OP_MASK_IMM5;
+           immed = GET_OP (l, IMM5);
            info->insn_type = dis_dref;
            info->data_size = 1;
            break;
          case 'H':
            nbits = 5;
            shift = 1;
-           immed = (l >> MIPS16OP_SH_IMM5) & MIPS16OP_MASK_IMM5;
+           immed = GET_OP (l, IMM5);
            info->insn_type = dis_dref;
            info->data_size = 2;
            break;
          case 'W':
            nbits = 5;
            shift = 2;
-           immed = (l >> MIPS16OP_SH_IMM5) & MIPS16OP_MASK_IMM5;
+           immed = GET_OP (l, IMM5);
            if ((op->pinfo & MIPS16_INSN_READ_PC) == 0
                && (op->pinfo & MIPS16_INSN_READ_SP) == 0)
              {
@@ -1764,27 +1710,27 @@ print_mips16_insn_arg (char type,
          case 'D':
            nbits = 5;
            shift = 3;
-           immed = (l >> MIPS16OP_SH_IMM5) & MIPS16OP_MASK_IMM5;
+           immed = GET_OP (l, IMM5);
            info->insn_type = dis_dref;
            info->data_size = 8;
            break;
          case 'j':
            nbits = 5;
-           immed = (l >> MIPS16OP_SH_IMM5) & MIPS16OP_MASK_IMM5;
+           immed = GET_OP (l, IMM5);
            signedp = 1;
            break;
          case '6':
            nbits = 6;
-           immed = (l >> MIPS16OP_SH_IMM6) & MIPS16OP_MASK_IMM6;
+           immed = GET_OP (l, IMM6);
            break;
          case '8':
            nbits = 8;
-           immed = (l >> MIPS16OP_SH_IMM8) & MIPS16OP_MASK_IMM8;
+           immed = GET_OP (l, IMM8);
            break;
          case 'V':
            nbits = 8;
            shift = 2;
-           immed = (l >> MIPS16OP_SH_IMM8) & MIPS16OP_MASK_IMM8;
+           immed = GET_OP (l, IMM8);
            /* FIXME: This might be lw, or it might be addiu to $sp or
                $pc.  We assume it's load.  */
            info->insn_type = dis_dref;
@@ -1793,36 +1739,36 @@ print_mips16_insn_arg (char type,
          case 'C':
            nbits = 8;
            shift = 3;
-           immed = (l >> MIPS16OP_SH_IMM8) & MIPS16OP_MASK_IMM8;
+           immed = GET_OP (l, IMM8);
            info->insn_type = dis_dref;
            info->data_size = 8;
            break;
          case 'U':
            nbits = 8;
-           immed = (l >> MIPS16OP_SH_IMM8) & MIPS16OP_MASK_IMM8;
+           immed = GET_OP (l, IMM8);
            extu = 1;
            break;
          case 'k':
            nbits = 8;
-           immed = (l >> MIPS16OP_SH_IMM8) & MIPS16OP_MASK_IMM8;
+           immed = GET_OP (l, IMM8);
            signedp = 1;
            break;
          case 'K':
            nbits = 8;
            shift = 3;
-           immed = (l >> MIPS16OP_SH_IMM8) & MIPS16OP_MASK_IMM8;
+           immed = GET_OP (l, IMM8);
            signedp = 1;
            break;
          case 'p':
            nbits = 8;
-           immed = (l >> MIPS16OP_SH_IMM8) & MIPS16OP_MASK_IMM8;
+           immed = GET_OP (l, IMM8);
            signedp = 1;
            pcrel = 1;
            branch = 1;
            break;
          case 'q':
            nbits = 11;
-           immed = (l >> MIPS16OP_SH_IMM11) & MIPS16OP_MASK_IMM11;
+           immed = GET_OP (l, IMM11);
            signedp = 1;
            pcrel = 1;
            branch = 1;
@@ -1830,7 +1776,7 @@ print_mips16_insn_arg (char type,
          case 'A':
            nbits = 8;
            shift = 2;
-           immed = (l >> MIPS16OP_SH_IMM8) & MIPS16OP_MASK_IMM8;
+           immed = GET_OP (l, IMM8);
            pcrel = 1;
            /* FIXME: This can be lw or la.  We assume it is lw.  */
            info->insn_type = dis_dref;
@@ -1839,7 +1785,7 @@ print_mips16_insn_arg (char type,
          case 'B':
            nbits = 5;
            shift = 3;
-           immed = (l >> MIPS16OP_SH_IMM5) & MIPS16OP_MASK_IMM5;
+           immed = GET_OP (l, IMM5);
            pcrel = 1;
            info->insn_type = dis_dref;
            info->data_size = 8;
@@ -1847,7 +1793,7 @@ print_mips16_insn_arg (char type,
          case 'E':
            nbits = 5;
            shift = 2;
-           immed = (l >> MIPS16OP_SH_IMM5) & MIPS16OP_MASK_IMM5;
+           immed = GET_OP (l, IMM5);
            pcrel = 1;
            break;
          default:
@@ -1877,7 +1823,7 @@ print_mips16_insn_arg (char type,
          }
 
        if (! pcrel)
-         (*info->fprintf_func) (info->stream, "%d", immed);
+         infprintf (is, "%d", immed);
        else
          {
            bfd_vma baseaddr;
@@ -1955,51 +1901,43 @@ print_mips16_insn_arg (char type,
 
        need_comma = 0;
 
-       l = (l >> MIPS16OP_SH_IMM6) & MIPS16OP_MASK_IMM6;
+       l = GET_OP (l, IMM6);
 
        amask = (l >> 3) & 7;
 
        if (amask > 0 && amask < 5)
          {
-           (*info->fprintf_func) (info->stream, "%s", mips_gpr_names[4]);
+           infprintf (is, "%s", mips_gpr_names[4]);
            if (amask > 1)
-             (*info->fprintf_func) (info->stream, "-%s",
-                                    mips_gpr_names[amask + 3]);
+             infprintf (is, "-%s", mips_gpr_names[amask + 3]);
            need_comma = 1;
          }
 
        smask = (l >> 1) & 3;
        if (smask == 3)
          {
-           (*info->fprintf_func) (info->stream, "%s??",
-                                  need_comma ? "," : "");
+           infprintf (is, "%s??", need_comma ? "," : "");
            need_comma = 1;
          }
        else if (smask > 0)
          {
-           (*info->fprintf_func) (info->stream, "%s%s",
-                                  need_comma ? "," : "",
-                                  mips_gpr_names[16]);
+           infprintf (is, "%s%s", need_comma ? "," : "", mips_gpr_names[16]);
            if (smask > 1)
-             (*info->fprintf_func) (info->stream, "-%s",
-                                    mips_gpr_names[smask + 15]);
+             infprintf (is, "-%s", mips_gpr_names[smask + 15]);
            need_comma = 1;
          }
 
        if (l & 1)
          {
-           (*info->fprintf_func) (info->stream, "%s%s",
-                                  need_comma ? "," : "",
-                                  mips_gpr_names[31]);
+           infprintf (is, "%s%s", need_comma ? "," : "", mips_gpr_names[31]);
            need_comma = 1;
          }
 
        if (amask == 5 || amask == 6)
          {
-           (*info->fprintf_func) (info->stream, "%s$f0",
-                                  need_comma ? "," : "");
+           infprintf (is, "%s$f0", need_comma ? "," : "");
            if (amask == 6)
-             (*info->fprintf_func) (info->stream, "-$f1");
+             infprintf (is, "-$f1");
          }
       }
       break;
@@ -2036,10 +1974,9 @@ print_mips16_insn_arg (char type,
         }
 
       if (args > 0) {
-          (*info->fprintf_func) (info->stream, "%s", mips_gpr_names[4]);
+         infprintf (is, "%s", mips_gpr_names[4]);
           if (args > 1)
-            (*info->fprintf_func) (info->stream, "-%s",
-                                   mips_gpr_names[4 + args - 1]);
+           infprintf (is, "-%s", mips_gpr_names[4 + args - 1]);
           need_comma = 1;
       }
 
@@ -2047,12 +1984,10 @@ print_mips16_insn_arg (char type,
       if (framesz == 0 && !use_extend)
         framesz = 128;
 
-      (*info->fprintf_func) (info->stream, "%s%d", 
-                             need_comma ? "," : "",
-                             framesz);
+      infprintf (is, "%s%d", need_comma ? "," : "", framesz);
 
       if (l & 0x40)                   /* $ra */
-        (*info->fprintf_func) (info->stream, ",%s", mips_gpr_names[31]);
+       infprintf (is, ",%s", mips_gpr_names[31]);
 
       nsreg = (l >> 24) & 0x7;
       smask = 0;
@@ -2068,34 +2003,32 @@ print_mips16_insn_arg (char type,
         {
           if (smask & (1 << i))
             {
-              (*info->fprintf_func) (info->stream, ",%s",
-                                     mips_gpr_names[i == 8 ? 30 : (16 + i)]);
+             infprintf (is, ",%s", mips_gpr_names[i == 8 ? 30 : (16 + i)]);
               /* Skip over string of set bits.  */
               for (j = i; smask & (2 << j); j++)
                 continue;
               if (j > i)
-                (*info->fprintf_func) (info->stream, "-%s",
-                                       mips_gpr_names[j == 8 ? 30 : (16 + j)]);
+               infprintf (is, "-%s", mips_gpr_names[j == 8 ? 30 : (16 + j)]);
               i = j + 1;
             }
         }
 
       /* Statics $ax - $a3.  */
       if (statics == 1)
-        (*info->fprintf_func) (info->stream, ",%s", mips_gpr_names[7]);
+       infprintf (is, ",%s", mips_gpr_names[7]);
       else if (statics > 0) 
-        (*info->fprintf_func) (info->stream, ",%s-%s", 
-                               mips_gpr_names[7 - statics + 1],
-                               mips_gpr_names[7]);
+       infprintf (is, ",%s-%s",
+                  mips_gpr_names[7 - statics + 1],
+                  mips_gpr_names[7]);
       }
       break;
 
     default:
       /* xgettext:c-format */
-      (*info->fprintf_func)
-       (info->stream,
-        _("# internal disassembler error, unrecognised modifier (%c)"),
-        type);
+      infprintf (is,
+                _("# internal disassembler error, "
+                  "unrecognised modifier (%c)"),
+                type);
       abort ();
     }
 }
@@ -2105,6 +2038,7 @@ print_mips16_insn_arg (char type,
 static int
 print_insn_mips16 (bfd_vma memaddr, struct disassemble_info *info)
 {
+  const fprintf_ftype infprintf = info->fprintf_func;
   int status;
   bfd_byte buffer[2];
   int length;
@@ -2112,6 +2046,7 @@ print_insn_mips16 (bfd_vma memaddr, struct disassemble_info *info)
   bfd_boolean use_extend;
   int extend = 0;
   const struct mips_opcode *op, *opend;
+  void *is = info->stream;
 
   info->bytes_per_chunk = 2;
   info->display_endian = info->endian;
@@ -2148,8 +2083,7 @@ print_insn_mips16 (bfd_vma memaddr, struct disassemble_info *info)
       status = (*info->read_memory_func) (memaddr, buffer, 2, info);
       if (status != 0)
        {
-         (*info->fprintf_func) (info->stream, "extend 0x%x",
-                                (unsigned int) extend);
+         infprintf (is, "extend 0x%x", (unsigned int) extend);
          (*info->memory_error_func) (status, memaddr, info);
          return -1;
        }
@@ -2162,8 +2096,7 @@ print_insn_mips16 (bfd_vma memaddr, struct disassemble_info *info)
       /* Check for an extend opcode followed by an extend opcode.  */
       if ((insn & 0xf800) == 0xf000)
        {
-         (*info->fprintf_func) (info->stream, "extend 0x%x",
-                                (unsigned int) extend);
+         infprintf (is, "extend 0x%x", (unsigned int) extend);
          info->insn_type = dis_noninsn;
          return length;
        }
@@ -2186,8 +2119,7 @@ print_insn_mips16 (bfd_vma memaddr, struct disassemble_info *info)
            {
              if (use_extend)
                {
-                 (*info->fprintf_func) (info->stream, "extend 0x%x",
-                                        (unsigned int) extend);
+                 infprintf (is, "extend 0x%x", (unsigned int) extend);
                  info->insn_type = dis_noninsn;
                  return length - 2;
                }
@@ -2209,16 +2141,15 @@ print_insn_mips16 (bfd_vma memaddr, struct disassemble_info *info)
                }
            }
 
-         (*info->fprintf_func) (info->stream, "%s", op->name);
+         infprintf (is, "%s", op->name);
          if (op->args[0] != '\0')
-           (*info->fprintf_func) (info->stream, "\t");
+           infprintf (is, "\t");
 
          for (s = op->args; *s != '\0'; s++)
            {
              if (*s == ','
                  && s[1] == 'w'
-                 && (((insn >> MIPS16OP_SH_RX) & MIPS16OP_MASK_RX)
-                     == ((insn >> MIPS16OP_SH_RY) & MIPS16OP_MASK_RY)))
+                 && GET_OP (insn, RX) == GET_OP (insn, RY))
                {
                  /* Skip the register and the comma.  */
                  ++s;
@@ -2226,8 +2157,7 @@ print_insn_mips16 (bfd_vma memaddr, struct disassemble_info *info)
                }
              if (*s == ','
                  && s[1] == 'v'
-                 && (((insn >> MIPS16OP_SH_RZ) & MIPS16OP_MASK_RZ)
-                     == ((insn >> MIPS16OP_SH_RX) & MIPS16OP_MASK_RX)))
+                 && GET_OP (insn, RZ) == GET_OP (insn, RX))
                {
                  /* Skip the register and the comma.  */
                  ++s;
@@ -2254,10 +2184,12 @@ print_insn_mips16 (bfd_vma memaddr, struct disassemble_info *info)
          return length;
        }
     }
+#undef GET_OP_S
+#undef GET_OP
 
   if (use_extend)
-    (*info->fprintf_func) (info->stream, "0x%x", extend | 0xf000);
-  (*info->fprintf_func) (info->stream, "0x%x", insn);
+    infprintf (is, "0x%x", extend | 0xf000);
+  infprintf (is, "0x%x", insn);
   info->insn_type = dis_noninsn;
 
   return length;
@@ -2368,6 +2300,9 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
 
 #define GET_OP(insn, field) \
   (((insn) >> MICROMIPSOP_SH_##field) & MICROMIPSOP_MASK_##field)
+#define GET_OP_S(insn, field) \
+  ((GET_OP (insn, field) ^ ((MICROMIPSOP_MASK_##field >> 1) + 1)) \
+   - ((MICROMIPSOP_MASK_##field >> 1) + 1))
   opend = micromips_opcodes + bfd_micromips_num_opcodes;
   for (op = micromips_opcodes; op < opend; op++)
     {
@@ -2394,33 +2329,64 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
                  break;
 
                case '.':
-                 delta = GET_OP (insn, OFFSET10);
-                 if (delta & 0x200)
-                   delta |= ~0x3ff;
-                 infprintf (is, "%d", delta);
+                 infprintf (is, "%d", GET_OP_S (insn, OFFSET10));
                  break;
 
                case '1':
-                 infprintf (is, "0x%lx", GET_OP (insn, STYPE));
+                 infprintf (is, "0x%x", GET_OP (insn, STYPE));
+                 break;
+
+               case '2':
+                 infprintf (is, "0x%x", GET_OP (insn, BP));
+                 break;
+
+               case '3':
+                 infprintf (is, "0x%x", GET_OP (insn, SA3));
+                 break;
+
+               case '4':
+                 infprintf (is, "0x%x", GET_OP (insn, SA4));
+                 break;
+
+               case '5':
+                 infprintf (is, "0x%x", GET_OP (insn, IMM8));
+                 break;
+
+               case '6':
+                 infprintf (is, "0x%x", GET_OP (insn, RS));
+                 break;
+
+               case '7':
+                 infprintf (is, "$ac%d", GET_OP (insn, DSPACC));
+                 break;
+
+               case '8':
+                 infprintf (is, "0x%x", GET_OP (insn, WRDSP));
+                 break;
+
+               case '0': /* DSP 6-bit signed immediate in bit 16.  */
+                 delta = (GET_OP (insn, DSPSFT) ^ 0x20) - 0x20;
+                 infprintf (is, "%d", delta);
                  break;
 
                case '<':
-                 infprintf (is, "0x%lx", GET_OP (insn, SHAMT));
+                 infprintf (is, "0x%x", GET_OP (insn, SHAMT));
                  break;
 
                case '\\':
-                 infprintf (is, "0x%lx", GET_OP (insn, 3BITPOS));
+                 infprintf (is, "0x%x", GET_OP (insn, 3BITPOS));
+                 break;
+
+               case '^':
+                 infprintf (is, "0x%x", GET_OP (insn, RD));
                  break;
 
                case '|':
-                 infprintf (is, "0x%lx", GET_OP (insn, TRAP));
+                 infprintf (is, "0x%x", GET_OP (insn, TRAP));
                  break;
 
                case '~':
-                 delta = GET_OP (insn, OFFSET12);
-                 if (delta & 0x800)
-                   delta |= ~0x7ff;
-                 infprintf (is, "%d", delta);
+                 infprintf (is, "%d", GET_OP_S (insn, OFFSET12));
                  break;
 
                case 'a':
@@ -2429,7 +2395,7 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
                                    | (GET_OP (insn, TARGET) << 2));
                  else
                    info->target = (((memaddr + 4) & ~(bfd_vma) 0x07ffffff)
-                                   | ((GET_OP (insn, TARGET)) << 1));
+                                   | (GET_OP (insn, TARGET) << 1));
                  /* For gdb disassembler, force odd address on jalx.  */
                  if (info->flavour == bfd_target_unknown_flavour
                      && strcmp (op->name, "jalx") == 0)
@@ -2445,7 +2411,7 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
                  break;
 
                case 'c':
-                 infprintf (is, "0x%lx", GET_OP (insn, CODE));
+                 infprintf (is, "0x%x", GET_OP (insn, CODE));
                  break;
 
                case 'd':
@@ -2453,18 +2419,17 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
                  break;
 
                case 'h':
-                 infprintf (is, "0x%lx", GET_OP (insn, PREFX));
+                 infprintf (is, "0x%x", GET_OP (insn, PREFX));
                  break;
 
                case 'i':
                case 'u':
-                 infprintf (is, "0x%lx", GET_OP (insn, IMMEDIATE));
+                 infprintf (is, "0x%x", GET_OP (insn, IMMEDIATE));
                  break;
 
                case 'j': /* Same as i, but sign-extended.  */
                case 'o':
-                 delta = (GET_OP (insn, DELTA) ^ 0x8000) - 0x8000;
-                 infprintf (is, "%d", delta);
+                 infprintf (is, "%d", GET_OP_S (insn, DELTA));
                  break;
 
                case 'k':
@@ -2506,13 +2471,13 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
 
                case 'p':
                  /* Sign-extend the displacement.  */
-                 delta = (GET_OP (insn, DELTA) ^ 0x8000) - 0x8000;
+                 delta = GET_OP_S (insn, DELTA);
                  info->target = (delta << 1) + memaddr + length;
                  (*info->print_address_func) (info->target, info);
                  break;
 
                case 'q':
-                 infprintf (is, "0x%lx", GET_OP (insn, CODE2));
+                 infprintf (is, "0x%x", GET_OP (insn, CODE2));
                  break;
 
                case 't':
@@ -2528,12 +2493,17 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
                  infprintf (is, "%s", mips_gpr_names[0]);
                  break;
 
+               case '@': /* DSP 10-bit signed immediate in bit 16.  */
+                 delta = (GET_OP (insn, IMM10) ^ 0x200) - 0x200;
+                 infprintf (is, "%d", delta);
+                 break;
+
                case 'B':
-                 infprintf (is, "0x%lx", GET_OP (insn, CODE10));
+                 infprintf (is, "0x%x", GET_OP (insn, CODE10));
                  break;
 
                case 'C':
-                 infprintf (is, "0x%lx", GET_OP (insn, COPZ));
+                 infprintf (is, "0x%x", GET_OP (insn, COPZ));
                  break;
 
                case 'D':
@@ -2548,7 +2518,7 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
                    'T' format.  Therefore, until we gain understanding of
                    cp2 register names, we can simply print the register
                    numbers.  */
-                 infprintf (is, "$%ld", GET_OP (insn, RT));
+                 infprintf (is, "$%d", GET_OP (insn, RT));
                  break;
 
                case 'G':
@@ -2570,13 +2540,13 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
                      infprintf (is, "%s", mips_cp0_names[GET_OP (insn, RS)]);
                      break;
                    default:
-                     infprintf (is, "$%ld", GET_OP (insn, RS));
+                     infprintf (is, "$%d", GET_OP (insn, RS));
                      break;
                    }
                  break;
 
                case 'H':
-                 infprintf (is, "%ld", GET_OP (insn, SEL));
+                 infprintf (is, "%d", GET_OP (insn, SEL));
                  break;
 
                case 'K':
@@ -2584,13 +2554,13 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
                  break;
 
                case 'M':
-                 infprintf (is, "$fcc%ld", GET_OP (insn, CCC));
+                 infprintf (is, "$fcc%d", GET_OP (insn, CCC));
                  break;
 
                case 'N':
                  infprintf (is,
                           (op->pinfo & (FP_D | FP_S)) != 0
-                          ? "$fcc%ld" : "$cc%ld",
+                          ? "$fcc%d" : "$cc%d",
                           GET_OP (insn, BCC));
                  break;
 
@@ -2775,7 +2745,7 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
 
                    case 'A':
                      /* Sign-extend the immediate.  */
-                     immed = ((GET_OP (insn, IMMA) ^ 0x40) - 0x40) << 2;
+                     immed = GET_OP_S (insn, IMMA) << 2;
                      infprintf (is, "%d", immed);
                      break;
 
@@ -2786,19 +2756,19 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
 
                    case 'C':
                      immed = micromips_imm_c_map[GET_OP (insn, IMMC)];
-                     infprintf (is, "0x%lx", immed);
+                     infprintf (is, "0x%x", immed);
                      break;
 
                    case 'D':
                      /* Sign-extend the displacement.  */
-                     delta = (GET_OP (insn, IMMD) ^ 0x200) - 0x200;
+                     delta = GET_OP_S (insn, IMMD);
                      info->target = (delta << 1) + memaddr + length;
                      (*info->print_address_func) (info->target, info);
                      break;
 
                    case 'E':
                      /* Sign-extend the displacement.  */
-                     delta = (GET_OP (insn, IMME) ^ 0x40) - 0x40;
+                     delta = GET_OP_S (insn, IMME);
                      info->target = (delta << 1) + memaddr + length;
                      (*info->print_address_func) (info->target, info);
                      break;
@@ -2866,8 +2836,7 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
 
                    case 'Q':
                      /* Sign-extend the immediate.  */
-                     immed = (GET_OP (insn, IMMQ) ^ 0x400000) - 0x400000;
-                     immed <<= 2;
+                     immed = GET_OP_S (insn, IMMQ) << 2;
                      infprintf (is, "%d", immed);
                      break;
 
@@ -2883,16 +2852,15 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
 
                    case 'X':
                      /* Sign-extend the immediate.  */
-                     immed = (GET_OP (insn, IMMX) ^ 0x8) - 0x8;
+                     immed = GET_OP_S (insn, IMMX);
                      infprintf (is, "%d", immed);
                      break;
 
                    case 'Y':
                      /* Sign-extend the immediate.  */
-                     immed = (GET_OP (insn, IMMY) ^ 0x100) - 0x100;
-                     if (immed >= -2 && immed <= 1)
-                       immed ^= 0x100;
-                     immed = immed << 2;
+                     immed = GET_OP_S (insn, IMMY) << 2;
+                     if ((unsigned int) (immed + 8) < 16)
+                       immed ^= 0x400;
                      infprintf (is, "%d", immed);
                      break;
 
@@ -2943,6 +2911,7 @@ print_insn_micromips (bfd_vma memaddr, struct disassemble_info *info)
          return length;
        }
     }
+#undef GET_OP_S
 #undef GET_OP
 
   infprintf (is, "0x%x", insn);
@@ -2973,6 +2942,9 @@ is_compressed_mode_p (struct disassemble_info *info)
       if (bfd_asymbol_flavour (info->symtab[pos]) != bfd_target_elf_flavour)
        continue;
 
+      if (info->symtab[pos]->section != info->section)
+       continue;
+
       symbol = (elf_symbol_type *) info->symtab[pos];
       if ((!micromips_ase
           && ELF_ST_IS_MIPS16 (symbol->internal_elf_sym.st_other))
@@ -3024,12 +2996,12 @@ _print_insn_mips (bfd_vma memaddr,
   status = (*info->read_memory_func) (memaddr, buffer, INSNLEN, info);
   if (status == 0)
     {
-      unsigned long insn;
+      int insn;
 
       if (endianness == BFD_ENDIAN_BIG)
-       insn = (unsigned long) bfd_getb32 (buffer);
+       insn = bfd_getb32 (buffer);
       else
-       insn = (unsigned long) bfd_getl32 (buffer);
+       insn = bfd_getl32 (buffer);
 
       return print_insn_mips (memaddr, insn, info);
     }