2002-08-21 Elena Zannoni <ezannoni@redhat.com>
[deliverable/binutils-gdb.git] / opcodes / sparc-dis.c
index 1dc2e352c310f6d6adbe1acaf7b01020037fdde9..47ebb313744842a62aaf005a2c6dc5d88013ef0f 100644 (file)
@@ -1,5 +1,6 @@
 /* Print SPARC instructions.
-   Copyright 1989, 1991, 1992, 1993, 1995 Free Software Foundation, Inc.
+   Copyright 1989, 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999,
+   2000, 2002 Free Software Foundation, Inc.
 
 This program is free software; you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
@@ -15,11 +16,25 @@ You should have received a copy of the GNU General Public License
 along with this program; if not, write to the Free Software
 Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
-#include "ansidecl.h"
+#include <stdio.h>
+
+#include "sysdep.h"
 #include "opcode/sparc.h"
 #include "dis-asm.h"
 #include "libiberty.h"
-#include <string.h>
+#include "opintl.h"
+
+/* Bitmask of v9 architectures.  */
+#define MASK_V9 ((1 << SPARC_OPCODE_ARCH_V9) \
+                | (1 << SPARC_OPCODE_ARCH_V9A) \
+                | (1 << SPARC_OPCODE_ARCH_V9B))
+/* 1 if INSN is for v9 only.  */
+#define V9_ONLY_P(insn) (! ((insn)->architecture & ~MASK_V9))
+/* 1 if INSN is for v9.  */
+#define V9_P(insn) (((insn)->architecture & MASK_V9) != 0)
+
+/* The sorted opcode table.  */
+static const struct sparc_opcode **sorted_opcodes;
 
 /* For faster lookup, after insns are sorted they are hashed.  */
 /* ??? I think there is room for even more improvement.  */
@@ -33,10 +48,15 @@ static int opcode_bits[4] = { 0x01c00000, 0x0, 0x01f80000, 0x01f80000 };
   ((((INSN) >> 24) & 0xc0) | (((INSN) & opcode_bits[((INSN) >> 30) & 3]) >> 19))
 struct opcode_hash {
   struct opcode_hash *next;
-  struct sparc_opcode *opcode;
+  const struct sparc_opcode *opcode;
 };
 static struct opcode_hash *opcode_hash_table[HASH_SIZE];
-static void build_hash_table ();
+
+static void build_hash_table
+  PARAMS ((const struct sparc_opcode **, struct opcode_hash **, int));
+static int is_delayed_branch PARAMS ((unsigned long));
+static int compare_opcodes PARAMS ((const PTR, const PTR));
+static int compute_arch_mask PARAMS ((unsigned long));
 
 /* Sign-extend a value which is N bits long.  */
 #define        SEX(value, bits) \
@@ -52,19 +72,16 @@ static  char *reg_names[] =
   "f8", "f9", "f10", "f11", "f12", "f13", "f14", "f15",        
   "f16", "f17", "f18", "f19", "f20", "f21", "f22", "f23",
   "f24", "f25", "f26", "f27", "f28", "f29", "f30", "f31",
-#ifndef NO_V9
   "f32", "f33", "f34", "f35", "f36", "f37", "f38", "f39",      
   "f40", "f41", "f42", "f43", "f44", "f45", "f46", "f47",      
   "f48", "f49", "f50", "f51", "f52", "f53", "f54", "f55",
   "f56", "f57", "f58", "f59", "f60", "f61", "f62", "f63",
 /* psr, wim, tbr, fpsr, cpsr are v8 only.  */
-#endif
   "y", "psr", "wim", "tbr", "pc", "npc", "fpsr", "cpsr"
 };
 
 #define        freg_names      (&reg_names[4 * 8])
 
-#ifndef NO_V9
 /* These are ordered according to there register number in
    rdpr and wrpr insns.  */
 static char *v9_priv_reg_names[] =
@@ -74,7 +91,14 @@ static char *v9_priv_reg_names[] =
   "wstate", "fq"
   /* "ver" - special cased */
 };
-#endif
+
+/* These are ordered according to there register number in
+   rd and wr insns (-16).  */
+static char *v9a_asr_reg_names[] =
+{
+  "pcr", "pic", "dcr", "gsr", "set_softint", "clear_softint",
+  "softint", "tick_cmpr", "sys_tick", "sys_tick_cmpr"
+};
 
 /* Macros used to extract instruction fields.  Not all fields have
    macros defined here, only those which are actually used.  */
@@ -84,14 +108,16 @@ static char *v9_priv_reg_names[] =
 #define X_LDST_I(i) (((i) >> 13) & 1)
 #define X_ASI(i) (((i) >> 5) & 0xff)
 #define X_RS2(i) (((i) >> 0) & 0x1f)
-#define X_IMM13(i) (((i) >> 0) & 0x1fff)
+#define X_IMM(i,n) (((i) >> 0) & ((1 << (n)) - 1))
+#define X_SIMM(i,n) SEX (X_IMM ((i), (n)), (n))
 #define X_DISP22(i) (((i) >> 0) & 0x3fffff)
 #define X_IMM22(i) X_DISP22 (i)
 #define X_DISP30(i) (((i) >> 0) & 0x3fffffff)
 
-#ifndef NO_V9
+/* These are for v9.  */
 #define X_DISP16(i) (((((i) >> 20) & 3) << 14) | (((i) >> 0) & 0x3fff))
-#endif
+#define X_DISP19(i) (((i) >> 0) & 0x7ffff)
+#define X_MEMBAR(i) ((i) & 0x7f)
 
 /* Here is the union which was used to extract instruction fields
    before the shift and mask macros were written.
@@ -131,7 +157,6 @@ static char *v9_priv_reg_names[] =
           #define      disp22  branch.DISP22
           #define      imm22   disp22
         } branch;
-        #ifndef NO_V9
        struct
         {
           unsigned int anop:2;
@@ -144,7 +169,6 @@ static char *v9_priv_reg_names[] =
           unsigned int _rs1:5;
           unsigned int DISP16LO:14;
         } branch16;
-        #endif
        struct
         {
           unsigned int anop:2;
@@ -164,7 +188,7 @@ is_delayed_branch (insn)
 
   for (op = opcode_hash_table[HASH_INSN (insn)]; op; op = op->next)
     {
-      CONST struct sparc_opcode *opcode = op->opcode;
+      const struct sparc_opcode *opcode = op->opcode;
       if ((opcode->match & insn) == opcode->match
          && (opcode->lose & insn) == 0)
        return (opcode->flags & F_DELAYED);
@@ -172,15 +196,11 @@ is_delayed_branch (insn)
   return 0;
 }
 
-/* Nonzero of opcode table has been initialized.  */
-static int opcodes_initialized = 0;
-
-/* Nonzero of the current architecture is sparc64.
-   This is kept in a global because compare_opcodes uses it.  */
-static int sparc64_p;
-
 /* extern void qsort (); */
-static int compare_opcodes ();
+
+/* Records current mask of SPARC_OPCODE_ARCH_FOO values, used to pass value
+   to compare_opcodes.  */
+static unsigned int current_arch_mask;
 
 /* Print one instruction from MEMADDR on INFO->STREAM.
 
@@ -190,22 +210,39 @@ static int compare_opcodes ();
    displacement to that register, or it is an `add' or `or' instruction
    on that register.  */
 
-static int
-print_insn (memaddr, info)
+int
+print_insn_sparc (memaddr, info)
      bfd_vma memaddr;
      disassemble_info *info;
 {
   FILE *stream = info->stream;
   bfd_byte buffer[4];
   unsigned long insn;
-  register unsigned int i;
   register struct opcode_hash *op;
-
-  if (!opcodes_initialized)
+  /* Nonzero of opcode table has been initialized.  */
+  static int opcodes_initialized = 0;
+  /* bfd mach number of last call.  */
+  static unsigned long current_mach = 0;
+  bfd_vma (*getword) PARAMS ((const unsigned char *));
+
+  if (!opcodes_initialized
+      || info->mach != current_mach)
     {
-      qsort ((char *) sparc_opcodes, NUMOPCODES,
-            sizeof (sparc_opcodes[0]), compare_opcodes);
-      build_hash_table (sparc_opcodes, opcode_hash_table, NUMOPCODES);
+      int i;
+
+      current_arch_mask = compute_arch_mask (info->mach);
+
+      if (!opcodes_initialized)
+       sorted_opcodes = (const struct sparc_opcode **)
+         xmalloc (sparc_num_opcodes * sizeof (struct sparc_opcode *));
+      /* Reset the sorted table so we can resort it.  */
+      for (i = 0; i < sparc_num_opcodes; ++i)
+       sorted_opcodes[i] = &sparc_opcodes[i];
+      qsort ((char *) sorted_opcodes, sparc_num_opcodes,
+            sizeof (sorted_opcodes[0]), compare_opcodes);
+
+      build_hash_table (sorted_opcodes, opcode_hash_table, sparc_num_opcodes);
+      current_mach = info->mach;
       opcodes_initialized = 1;
     }
 
@@ -219,7 +256,14 @@ print_insn (memaddr, info)
       }
   }
 
-  insn = bfd_getb32 (buffer);
+  /* On SPARClite variants such as DANlite (sparc86x), instructions
+     are always big-endian even when the machine is in little-endian mode. */
+  if (info->endian == BFD_ENDIAN_BIG || info->mach == bfd_mach_sparc_sparclite)
+    getword = bfd_getb32;
+  else
+    getword = bfd_getl32;
+
+  insn = getword (buffer);
 
   info->insn_info_valid = 1;                   /* We do return this info */
   info->insn_type = dis_nonbranch;             /* Assume non branch insn */
@@ -228,7 +272,11 @@ print_insn (memaddr, info)
 
   for (op = opcode_hash_table[HASH_INSN (insn)]; op; op = op->next)
     {
-      CONST struct sparc_opcode *opcode = op->opcode;
+      const struct sparc_opcode *opcode = op->opcode;
+
+      /* If the insn isn't supported by the current architecture, skip it.  */
+      if (! (opcode->architecture & current_arch_mask))
+       continue;
 
       if ((opcode->match & insn) == opcode->match
          && (opcode->lose & insn) == 0)
@@ -236,6 +284,7 @@ print_insn (memaddr, info)
          /* Nonzero means that we have found an instruction which has
             the effect of adding or or'ing the imm13 field to rs1.  */
          int imm_added_to_rs1 = 0;
+         int imm_ored_to_rs1 = 0;
 
          /* Nonzero means that we have found a plus sign in the args
             field of the opcode table.  */
@@ -244,22 +293,26 @@ print_insn (memaddr, info)
          /* Nonzero means we have an annulled branch.  */
          int is_annulled = 0;
 
-         /* Do we have an `add' or `or' instruction where rs1 is the same
-            as rsd, and which has the i bit set?  */
-         if ((opcode->match == 0x80102000 || opcode->match == 0x80002000)
-         /*                      (or)                           (add)  */
-             && X_RS1 (insn) == X_RD (insn))
+         /* Do we have an `add' or `or' instruction combining an
+             immediate with rs1?  */
+         if (opcode->match == 0x80102000) /* or */
+           imm_ored_to_rs1 = 1;
+         if (opcode->match == 0x80002000) /* add */
            imm_added_to_rs1 = 1;
 
          if (X_RS1 (insn) != X_RD (insn)
              && strchr (opcode->args, 'r') != 0)
              /* Can't do simple format if source and dest are different.  */
              continue;
+         if (X_RS2 (insn) != X_RD (insn)
+             && strchr (opcode->args, 'O') != 0)
+             /* Can't do simple format if source and dest are different.  */
+             continue;
 
          (*info->fprintf_func) (stream, opcode->name);
 
          {
-           register CONST char *s;
+           register const char *s;
 
            if (opcode->args[0] != ',')
              (*info->fprintf_func) (stream, " ");
@@ -275,7 +328,6 @@ print_insn (memaddr, info)
                      is_annulled = 1;
                      ++s;
                      continue;
-#ifndef NO_V9
                    case 'N':
                      (*info->fprintf_func) (stream, "pn");
                      ++s;
@@ -285,7 +337,6 @@ print_insn (memaddr, info)
                      (*info->fprintf_func) (stream, "pt");
                      ++s;
                      continue;
-#endif /* NO_V9 */
 
                    default:
                      break;
@@ -315,6 +366,7 @@ print_insn (memaddr, info)
                    break;
 
                  case '2':
+                 case 'O':
                    reg (X_RS2 (insn));
                    break;
 
@@ -367,13 +419,22 @@ print_insn (memaddr, info)
 
                  case 'h':
                    (*info->fprintf_func) (stream, "%%hi(%#x)",
-                                          (0xFFFFFFFF
+                                          ((unsigned) 0xFFFFFFFF
                                            & ((int) X_IMM22 (insn) << 10)));
                    break;
 
-                 case 'i':
+                 case 'i':     /* 13 bit immediate */
+                 case 'I':     /* 11 bit immediate */
+                 case 'j':     /* 10 bit immediate */
                    {
-                     int imm = SEX (X_IMM13 (insn), 13);
+                     int imm;
+
+                     if (*s == 'i')
+                       imm = X_SIMM (insn, 13);
+                     else if (*s == 'I')
+                       imm = X_SIMM (insn, 11);
+                     else
+                       imm = X_SIMM (insn, 10);
 
                      /* Check to see whether we have a 1+i, and take
                         note of that fact.
@@ -392,27 +453,11 @@ print_insn (memaddr, info)
                    }
                    break;
 
-#ifndef NO_V9
-                 case 'I':     /* 11 bit immediate.  */
-                 case 'j':     /* 10 bit immediate.  */
+                 case 'X':     /* 5 bit unsigned immediate */
+                 case 'Y':     /* 6 bit unsigned immediate */
                    {
-                     int imm;
-
-                     if (*s == 'I')
-                       imm = SEX (X_IMM13 (insn), 11);
-                     else
-                       imm = SEX (X_IMM13 (insn), 10);
+                     int imm = X_IMM (insn, *s == 'X' ? 5 : 6);
 
-                     /* Check to see whether we have a 1+i, and take
-                        note of that fact.
-                        
-                        Note: because of the way we sort the table,
-                        we will be matching 1+i rather than i+1,
-                        so it is OK to assume that i is after +,
-                        not before it.  */
-                     if (found_plus)
-                       imm_added_to_rs1 = 1;
-                     
                      if (imm <= 9)
                        (info->fprintf_func) (stream, "%d", imm);
                      else
@@ -420,13 +465,41 @@ print_insn (memaddr, info)
                    }
                    break;
 
+                 case '3':
+                   (info->fprintf_func) (stream, "%d", X_IMM (insn, 3));
+                   break;
+
+                 case 'K':
+                   {
+                     int mask = X_MEMBAR (insn);
+                     int bit = 0x40, printed_one = 0;
+                     const char *name;
+
+                     if (mask == 0)
+                       (info->fprintf_func) (stream, "0");
+                     else
+                       while (bit)
+                         {
+                           if (mask & bit)
+                             {
+                               if (printed_one)
+                                 (info->fprintf_func) (stream, "|");
+                               name = sparc_decode_membar (bit);
+                               (info->fprintf_func) (stream, "%s", name);
+                               printed_one = 1;
+                             }
+                           bit >>= 1;
+                         }
+                     break;
+                   }
+
                  case 'k':
-                   info->target = memaddr + (SEX (X_DISP16 (insn), 16)) * 4;
+                   info->target = memaddr + SEX (X_DISP16 (insn), 16) * 4;
                    (*info->print_address_func) (info->target, info);
                    break;
 
                  case 'G':
-                   info->target = memaddr + (SEX (X_DISP22 (insn), 19)) * 4;
+                   info->target = memaddr + SEX (X_DISP19 (insn), 19) * 4;
                    (*info->print_address_func) (info->target, info);
                    break;
 
@@ -456,7 +529,7 @@ print_insn (memaddr, info)
                  case 'o':
                    (*info->fprintf_func) (stream, "%%asi");
                    break;
-
+                   
                  case 'W':
                    (*info->fprintf_func) (stream, "%%tick");
                    break;
@@ -482,9 +555,34 @@ print_insn (memaddr, info)
                    else
                      (*info->fprintf_func) (stream, "%%reserved");
                    break;
+
+                 case '/':
+                   if (X_RS1 (insn) < 16 || X_RS1 (insn) > 25)
+                     (*info->fprintf_func) (stream, "%%reserved");
+                   else
+                     (*info->fprintf_func) (stream, "%%%s",
+                                            v9a_asr_reg_names[X_RS1 (insn)-16]);
                    break;
-#endif /* NO_V9 */
 
+                 case '_':
+                   if (X_RD (insn) < 16 || X_RD (insn) > 25)
+                     (*info->fprintf_func) (stream, "%%reserved");
+                   else
+                     (*info->fprintf_func) (stream, "%%%s",
+                                            v9a_asr_reg_names[X_RD (insn)-16]);
+                   break;
+
+                 case '*':
+                   {
+                     const char *name = sparc_decode_prefetch (X_RD (insn));
+
+                     if (name)
+                       (*info->fprintf_func) (stream, "%s", name);
+                     else
+                       (*info->fprintf_func) (stream, "%d", X_RD (insn));
+                     break;
+                   }
+                   
                  case 'M':
                    (*info->fprintf_func) (stream, "%%asr%d", X_RS1 (insn));
                    break;
@@ -494,23 +592,30 @@ print_insn (memaddr, info)
                    break;
                    
                  case 'L':
-                   info->target = memaddr + X_DISP30 (insn) * 4;
+                   info->target = memaddr + SEX (X_DISP30 (insn), 30) * 4;
                    (*info->print_address_func) (info->target, info);
                    break;
 
                  case 'n':
                    (*info->fprintf_func)
-                     (stream, "%#x", (SEX (X_DISP22 (insn), 22)));
+                     (stream, "%#x", SEX (X_DISP22 (insn), 22));
                    break;
 
                  case 'l':
-                   info->target = memaddr + (SEX (X_DISP22 (insn), 22)) * 4;
+                   info->target = memaddr + SEX (X_DISP22 (insn), 22) * 4;
                    (*info->print_address_func) (info->target, info);
                    break;
 
                  case 'A':
-                   (*info->fprintf_func) (stream, "(%d)", X_ASI (insn));
-                   break;
+                   {
+                     const char *name = sparc_decode_asi (X_ASI (insn));
+
+                     if (name)
+                       (*info->fprintf_func) (stream, "%s", name);
+                     else
+                       (*info->fprintf_func) (stream, "(%d)", X_ASI (insn));
+                     break;
+                   }
 
                  case 'C':
                    (*info->fprintf_func) (stream, "%%csr");
@@ -549,6 +654,19 @@ print_insn (memaddr, info)
                  case 'y':
                    (*info->fprintf_func) (stream, "%%y");
                    break;
+
+                 case 'u':
+                 case 'U':
+                   {
+                     int val = *s == 'U' ? X_RS1 (insn) : X_RD (insn);
+                     const char *name = sparc_decode_sparclet_cpreg (val);
+
+                     if (name)
+                       (*info->fprintf_func) (stream, "%s", name);
+                     else
+                       (*info->fprintf_func) (stream, "%%cpreg(%d)", val);
+                     break;
+                   }
                  }
              }
          }
@@ -559,7 +677,7 @@ print_insn (memaddr, info)
             If so, attempt to print the result of the add or
             or (in this context add and or do the same thing)
             and its symbolic value.  */
-         if (imm_added_to_rs1)
+         if (imm_ored_to_rs1 || imm_added_to_rs1)
            {
              unsigned long prev_insn;
              int errcode;
@@ -567,7 +685,7 @@ print_insn (memaddr, info)
              errcode =
                (*info->read_memory_func)
                  (memaddr - 4, buffer, sizeof (buffer), info);
-             prev_insn = bfd_getb32 (buffer);
+             prev_insn = getword (buffer);
 
              if (errcode == 0)
                {
@@ -584,7 +702,7 @@ print_insn (memaddr, info)
                    {
                      errcode = (*info->read_memory_func)
                        (memaddr - 8, buffer, sizeof (buffer), info);
-                     prev_insn = bfd_getb32 (buffer);
+                     prev_insn = getword (buffer);
                    }
                }
 
@@ -598,8 +716,12 @@ print_insn (memaddr, info)
                    {
                      (*info->fprintf_func) (stream, "\t! ");
                      info->target = 
-                       (0xFFFFFFFF & (int) X_IMM22 (prev_insn) << 10)
-                       | SEX (X_IMM13 (insn), 13);
+                       ((unsigned) 0xFFFFFFFF
+                        & ((int) X_IMM22 (prev_insn) << 10));
+                     if (imm_added_to_rs1)
+                       info->target += X_SIMM (insn, 13);
+                     else
+                       info->target |= X_SIMM (insn, 13);
                      (*info->print_address_func) (info->target, info);
                      info->insn_type = dis_dref;
                      info->data_size = 4;  /* FIXME!!! */
@@ -625,54 +747,98 @@ print_insn (memaddr, info)
     }
 
   info->insn_type = dis_noninsn;       /* Mark as non-valid instruction */
-  (*info->fprintf_func) (stream, "%#8x", insn);
+  (*info->fprintf_func) (stream, _("unknown"));
   return sizeof (buffer);
 }
 
+/* Given BFD mach number, return a mask of SPARC_OPCODE_ARCH_FOO values.  */
+
+static int
+compute_arch_mask (mach)
+     unsigned long mach;
+{
+  switch (mach)
+    {
+    case 0 :
+    case bfd_mach_sparc :
+      return SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_V8);
+    case bfd_mach_sparc_sparclet :
+      return SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_SPARCLET);
+    case bfd_mach_sparc_sparclite :
+    case bfd_mach_sparc_sparclite_le :
+      /* sparclites insns are recognized by default (because that's how
+        they've always been treated, for better or worse).  Kludge this by
+        indicating generic v8 is also selected.  */
+      return (SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_SPARCLITE)
+             | SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_V8));
+    case bfd_mach_sparc_v8plus :
+    case bfd_mach_sparc_v9 :
+      return SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_V9);
+    case bfd_mach_sparc_v8plusa :
+    case bfd_mach_sparc_v9a :
+      return SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_V9A);
+    case bfd_mach_sparc_v8plusb :
+    case bfd_mach_sparc_v9b :
+      return SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_V9B);
+    }
+  abort ();
+}
+
 /* Compare opcodes A and B.  */
 
 static int
 compare_opcodes (a, b)
-     char *a, *b;
+     const PTR a;
+     const PTR b;
 {
-  struct sparc_opcode *op0 = (struct sparc_opcode *) a;
-  struct sparc_opcode *op1 = (struct sparc_opcode *) b;
+  struct sparc_opcode *op0 = * (struct sparc_opcode **) a;
+  struct sparc_opcode *op1 = * (struct sparc_opcode **) b;
   unsigned long int match0 = op0->match, match1 = op1->match;
   unsigned long int lose0 = op0->lose, lose1 = op1->lose;
   register unsigned int i;
 
+  /* If one (and only one) insn isn't supported by the current architecture,
+     prefer the one that is.  If neither are supported, but they're both for
+     the same architecture, continue processing.  Otherwise (both unsupported
+     and for different architectures), prefer lower numbered arch's (fudged
+     by comparing the bitmasks).  */
+  if (op0->architecture & current_arch_mask)
+    {
+      if (! (op1->architecture & current_arch_mask))
+       return -1;
+    }
+  else
+    {
+      if (op1->architecture & current_arch_mask)
+       return 1;
+      else if (op0->architecture != op1->architecture)
+       return op0->architecture - op1->architecture;
+    }
+
   /* If a bit is set in both match and lose, there is something
      wrong with the opcode table.  */
   if (match0 & lose0)
     {
-      fprintf (stderr, "Internal error:  bad sparc-opcode.h: \"%s\", %#.8lx, %#.8lx\n",
-              op0->name, match0, lose0);
+      fprintf
+       (stderr,
+        /* xgettext:c-format */
+        _("Internal error:  bad sparc-opcode.h: \"%s\", %#.8lx, %#.8lx\n"),
+        op0->name, match0, lose0);
       op0->lose &= ~op0->match;
       lose0 = op0->lose;
     }
 
   if (match1 & lose1)
     {
-      fprintf (stderr, "Internal error: bad sparc-opcode.h: \"%s\", %#.8lx, %#.8lx\n",
-              op1->name, match1, lose1);
+      fprintf
+       (stderr,
+        /* xgettext:c-format */
+        _("Internal error: bad sparc-opcode.h: \"%s\", %#.8lx, %#.8lx\n"),
+        op1->name, match1, lose1);
       op1->lose &= ~op1->match;
       lose1 = op1->lose;
     }
 
-  /* If the current architecture isn't sparc64, move v9 insns to the end.
-     Only do this when one isn't v9 and one is.  If both are v9 we still
-     need to properly sort them.
-     This must be done before checking match and lose.  */
-  if (!sparc64_p
-      && (op0->architecture == v9) != (op1->architecture == v9))
-    return (op0->architecture == v9) - (op1->architecture == v9);
-
-  /* If the current architecture is sparc64, move non-v9 insns to the end.
-     This must be done before checking match and lose.  */
-  if (sparc64_p
-      && (op0->flags & F_NOTV9) != (op1->flags & F_NOTV9))
-    return (op0->flags & F_NOTV9) - (op1->flags & F_NOTV9);
-
   /* Because the bits that are variable in one opcode are constant in
      another, it is important to order the opcodes in the right order.  */
   for (i = 0; i < 32; ++i)
@@ -710,12 +876,15 @@ compare_opcodes (a, b)
      better have the same opcode.  This is a sanity check on the table.  */
   i = strcmp (op0->name, op1->name);
   if (i)
+    {
       if (op0->flags & F_ALIAS) /* If they're both aliases, be arbitrary. */
-         return i;
+       return i;
       else
-         fprintf (stderr,
-                  "Internal error: bad sparc-opcode.h: \"%s\" == \"%s\"\n",
-                  op0->name, op1->name);
+       fprintf (stderr,
+                /* xgettext:c-format */
+                _("Internal error: bad sparc-opcode.h: \"%s\" == \"%s\"\n"),
+                op0->name, op1->name);
+    }
 
   /* Fewer arguments are preferred.  */
   {
@@ -727,8 +896,8 @@ compare_opcodes (a, b)
 
   /* Put 1+i before i+1.  */
   {
-    char *p0 = (char *) strchr(op0->args, '+');
-    char *p1 = (char *) strchr(op1->args, '+');
+    char *p0 = (char *) strchr (op0->args, '+');
+    char *p1 = (char *) strchr (op1->args, '+');
 
     if (p0 && p1)
       {
@@ -744,38 +913,51 @@ compare_opcodes (a, b)
       }
   }
 
+  /* Put 1,i before i,1.  */
+  {
+    int i0 = strncmp (op0->args, "i,1", 3) == 0;
+    int i1 = strncmp (op1->args, "i,1", 3) == 0;
+
+    if (i0 ^ i1)
+      return i0 - i1;
+  }
+
   /* They are, as far as we can tell, identical.
      Since qsort may have rearranged the table partially, there is
      no way to tell which one was first in the opcode table as
      written, so just say there are equal.  */
+  /* ??? This is no longer true now that we sort a vector of pointers,
+     not the table itself.  */
   return 0;
 }
 
-/* Build a hash table from the opcode table.  */
+/* Build a hash table from the opcode table.
+   OPCODE_TABLE is a sorted list of pointers into the opcode table.  */
 
 static void
-build_hash_table (table, hash_table, num_opcodes)
-     struct sparc_opcode *table;
+build_hash_table (opcode_table, hash_table, num_opcodes)
+     const struct sparc_opcode **opcode_table;
      struct opcode_hash **hash_table;
      int num_opcodes;
 {
   register int i;
   int hash_count[HASH_SIZE];
-  struct opcode_hash *hash_buf;
+  static struct opcode_hash *hash_buf = NULL;
 
   /* Start at the end of the table and work backwards so that each
      chain is sorted.  */
-  /*  ??? Do we really need to sort them now?  */
 
   memset (hash_table, 0, HASH_SIZE * sizeof (hash_table[0]));
   memset (hash_count, 0, HASH_SIZE * sizeof (hash_count[0]));
+  if (hash_buf != NULL)
+    free (hash_buf);
   hash_buf = (struct opcode_hash *) xmalloc (sizeof (struct opcode_hash) * num_opcodes);
   for (i = num_opcodes - 1; i >= 0; --i)
     {
-      register int hash = HASH_INSN (sparc_opcodes[i].match);
+      register int hash = HASH_INSN (opcode_table[i]->match);
       register struct opcode_hash *h = &hash_buf[i];
       h->next = hash_table[hash];
-      h->opcode = &sparc_opcodes[i];
+      h->opcode = opcode_table[i];
       hash_table[hash] = h;
       ++hash_count[hash];
     }
@@ -799,31 +981,3 @@ build_hash_table (table, hash_table, num_opcodes)
   }
 #endif
 }
-
-int
-print_insn_sparc (memaddr, info)
-     bfd_vma memaddr;
-     disassemble_info *info;
-{
-  /* It's not clear that we'll ever switch cpus in a running program.
-     It could theoretically happen in gdb so we handle it.
-     ??? There is currently a memory leak but it's not worth the trouble.  */
-  if (sparc64_p)
-    opcodes_initialized = 0;
-  sparc64_p = 0;
-  return print_insn (memaddr, info);
-}
-
-int
-print_insn_sparc64 (memaddr, info)
-     bfd_vma memaddr;
-     disassemble_info *info;
-{
-  /* It's not clear that we'll ever switch cpus in a running program.
-     It could theoretically happen in gdb so we handle it.
-     ??? There is currently a memory leak but it's not worth the trouble.  */
-  if (!sparc64_p)
-    opcodes_initialized = 0;
-  sparc64_p = 1;
-  return print_insn (memaddr, info);
-}
This page took 0.032441 seconds and 4 git commands to generate.