gdb: add target_ops::supports_displaced_step
[deliverable/binutils-gdb.git] / opcodes / sparc-dis.c
index 2d20e6067dd6c6f76f4a9a9082fca3f4a965b447..c9667f172b40226905a9642508611e4701a75809 100644 (file)
 /* Print SPARC instructions.
-   Copyright 1989, 1991, 1992, 1993 Free Software Foundation, Inc.
+   Copyright (C) 1989-2020 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
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   This file is part of the GNU opcodes library.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   This library is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3, or (at your option)
+   any later version.
 
-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., 675 Mass Ave, Cambridge, MA 02139, USA.  */
+   It is distributed in the hope that it will be useful, but WITHOUT
+   ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+   or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public
+   License for more details.
 
+   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., 51 Franklin Street - Fifth Floor, Boston,
+   MA 02110-1301, USA.  */
+
+#include "sysdep.h"
+#include <stdio.h>
 #include "opcode/sparc.h"
 #include "dis-asm.h"
-#include <string.h>
+#include "libiberty.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 << SPARC_OPCODE_ARCH_V9C) \
+                | (1 << SPARC_OPCODE_ARCH_V9D) \
+                | (1 << SPARC_OPCODE_ARCH_V9E) \
+                | (1 << SPARC_OPCODE_ARCH_V9V) \
+                | (1 << SPARC_OPCODE_ARCH_V9M) \
+                | (1 << SPARC_OPCODE_ARCH_M8))
+/* 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 sparc_opcode **sorted_opcodes;
+
+/* For faster lookup, after insns are sorted they are hashed.  */
+/* ??? I think there is room for even more improvement.  */
+
+#define HASH_SIZE 256
+/* It is important that we only look at insn code bits as that is how the
+   opcode table is hashed.  OPCODE_BITS is a table of valid bits for each
+   of the main types (0,1,2,3).  */
+static int opcode_bits[4] = { 0x01c00000, 0x0, 0x01f80000, 0x01f80000 };
+#define HASH_INSN(INSN) \
+  ((((INSN) >> 24) & 0xc0) | (((INSN) & opcode_bits[((INSN) >> 30) & 3]) >> 19))
+typedef struct sparc_opcode_hash
+{
+  struct sparc_opcode_hash *next;
+  const sparc_opcode *opcode;
+} sparc_opcode_hash;
+
+static sparc_opcode_hash *opcode_hash_table[HASH_SIZE];
+
+/* Sign-extend a value which is N bits long.  */
+#define        SEX(value, bits) \
+  ((int) (((value & ((1u << (bits - 1) << 1) - 1))     \
+          ^ (1u << (bits - 1))) - (1u << (bits - 1))))
 
 static  char *reg_names[] =
- { "g0", "g1", "g2", "g3", "g4", "g5", "g6", "g7",     
-  "o0", "o1", "o2", "o3", "o4", "o5", "sp", "o7",      
-  "l0", "l1", "l2", "l3", "l4", "l5", "l6", "l7",      
-  "i0", "i1", "i2", "i3", "i4", "i5", "fp", "i7",      
-  "f0", "f1", "f2", "f3", "f4", "f5", "f6", "f7",      
-  "f8", "f9", "f10", "f11", "f12", "f13", "f14", "f15",        
+{ "g0", "g1", "g2", "g3", "g4", "g5", "g6", "g7",
+  "o0", "o1", "o2", "o3", "o4", "o5", "sp", "o7",
+  "l0", "l1", "l2", "l3", "l4", "l5", "l6", "l7",
+  "i0", "i1", "i2", "i3", "i4", "i5", "fp", "i7",
+  "f0", "f1", "f2", "f3", "f4", "f5", "f6", "f7",
+  "f8", "f9", "f10", "f11", "f12", "f13", "f14", "f15",
   "f16", "f17", "f18", "f19", "f20", "f21", "f22", "f23",
   "f24", "f25", "f26", "f27", "f28", "f29", "f30", "f31",
-  "y", "psr", "wim", "tbr", "pc", "npc", "fpsr", "cpsr" };
+  "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.  */
+  "y", "psr", "wim", "tbr", "pc", "npc", "fpsr", "cpsr"
+};
 
 #define        freg_names      (&reg_names[4 * 8])
 
-/* FIXME--need to deal with byte order (probably using masking and
-   shifting rather than bitfields is easiest).  */
-
-union sparc_insn
-  {
-    unsigned long int code;
-    struct
-      {
-       unsigned int anop:2;
-#define        op      ldst.anop
-       unsigned int anrd:5;
-#define        rd      ldst.anrd
-       unsigned int op3:6;
-       unsigned int anrs1:5;
-#define        rs1     ldst.anrs1
-       unsigned int i:1;
-       unsigned int anasi:8;
-#define        asi     ldst.anasi
-       unsigned int anrs2:5;
-#define        rs2     ldst.anrs2
-#define        shcnt   rs2
-      } ldst;
-    struct
-      {
-       unsigned int anop:2, anrd:5, op3:6, anrs1:5, i:1;
-       unsigned int IMM13:13;
-#define        imm13   IMM13.IMM13
-      } IMM13;
-    struct
-      {
-       unsigned int anop:2;
-       unsigned int a:1;
-       unsigned int cond:4;
-       unsigned int op2:3;
-       unsigned int DISP22:22;
-#define        disp22  branch.DISP22
-      } branch;
-#ifndef NO_V9
-    struct
-      {
-       unsigned int _OP:2, _RD:5, op3:6, _RS1:5;
-       unsigned int DISP14:14;
-#define        disp14  DISP14.DISP14
-      } DISP14;
-    struct
-      {
-       unsigned int _OP:2;
-       unsigned int a:1;
-       unsigned int cond:4;
-       unsigned int op2:3;
-       unsigned int p:1;
-       unsigned int DISP21:21;
-#define        disp21  branch2.DISP21
-      } branch2;
-#endif /* NO_V9 */
-
-#define        imm22   disp22
-    struct
-      {
-       unsigned int anop:2;
-       unsigned int adisp30:30;
-#define        disp30  call.adisp30
-      } call;
-  };
+/* These are ordered according to there register number in
+   rdpr and wrpr insns.  */
+static char *v9_priv_reg_names[] =
+{
+  "tpc", "tnpc", "tstate", "tt", "tick", "tba", "pstate", "tl",
+  "pil", "cwp", "cansave", "canrestore", "cleanwin", "otherwin",
+  "wstate", "fq", "gl"
+  /* "ver" and "pmcdper" - special cased */
+};
+
+/* These are ordered according to there register number in
+   rdhpr and wrhpr insns.  */
+static char *v9_hpriv_reg_names[] =
+{
+  "hpstate", "htstate", "resv2", "hintp", "resv4", "htba", "hver",
+  "resv7", "resv8", "resv9", "resv10", "resv11", "resv12", "resv13",
+  "resv14", "resv15", "resv16", "resv17", "resv18", "resv19", "resv20",
+  "resv21", "resv22", "hmcdper", "hmcddfr", "resv25", "resv26", "hva_mask_nz",
+  "hstick_offset", "hstick_enable", "resv30", "hstick_cmpr"
+};
+
+/* These are ordered according to there register number in
+   rd and wr insns (-16).  */
+static char *v9a_asr_reg_names[] =
+{
+  "pcr", "pic", "dcr", "gsr", "softint_set", "softint_clear",
+  "softint", "tick_cmpr", "stick", "stick_cmpr", "cfr",
+  "pause", "mwait"
+};
+
+/* Macros used to extract instruction fields.  Not all fields have
+   macros defined here, only those which are actually used.  */
+
+#define X_RD(i)      (((i) >> 25) & 0x1f)
+#define X_RS1(i)     (((i) >> 14) & 0x1f)
+#define X_LDST_I(i)  (((i) >> 13) & 1)
+#define X_ASI(i)     (((i) >> 5) & 0xff)
+#define X_RS2(i)     (((i) >> 0) & 0x1f)
+#define X_RS3(i)     (((i) >> 9) & 0x1f)
+#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)
+#define X_IMM2(i)    (((i & 0x10) >> 3) | (i & 0x1))
+
+/* These are for v9.  */
+#define X_DISP16(i)  (((((i) >> 20) & 3) << 14) | (((i) >> 0) & 0x3fff))
+#define X_DISP10(i)  (((((i) >> 19) & 3) << 8) | (((i) >> 5) & 0xff))
+#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.
+
+   union sparc_insn
+     {
+       unsigned long int code;
+       struct
+        {
+          unsigned int anop:2;
+          #define      op      ldst.anop
+          unsigned int anrd:5;
+          #define      rd      ldst.anrd
+          unsigned int op3:6;
+          unsigned int anrs1:5;
+          #define      rs1     ldst.anrs1
+          unsigned int i:1;
+          unsigned int anasi:8;
+          #define      asi     ldst.anasi
+          unsigned int anrs2:5;
+          #define      rs2     ldst.anrs2
+          #define      shcnt   rs2
+        } ldst;
+       struct
+        {
+          unsigned int anop:2, anrd:5, op3:6, anrs1:5, i:1;
+          unsigned int IMM13:13;
+          #define      imm13   IMM13.IMM13
+        } IMM13;
+       struct
+        {
+          unsigned int anop:2;
+          unsigned int a:1;
+          unsigned int cond:4;
+          unsigned int op2:3;
+          unsigned int DISP22:22;
+          #define      disp22  branch.DISP22
+          #define      imm22   disp22
+        } branch;
+       struct
+        {
+          unsigned int anop:2;
+          unsigned int a:1;
+          unsigned int z:1;
+          unsigned int rcond:3;
+          unsigned int op2:3;
+          unsigned int DISP16HI:2;
+          unsigned int p:1;
+          unsigned int _rs1:5;
+          unsigned int DISP16LO:14;
+        } branch16;
+       struct
+        {
+          unsigned int anop:2;
+          unsigned int adisp30:30;
+          #define      disp30  call.adisp30
+        } call;
+     };  */
 
 /* Nonzero if INSN is the opcode for a delayed branch.  */
+
 static int
-is_delayed_branch (insn)
-     union sparc_insn insn;
+is_delayed_branch (unsigned long insn)
 {
-  unsigned int i;
+  sparc_opcode_hash *op;
+
+  for (op = opcode_hash_table[HASH_INSN (insn)]; op; op = op->next)
+    {
+      const sparc_opcode *opcode = op->opcode;
+
+      if ((opcode->match & insn) == opcode->match
+         && (opcode->lose & insn) == 0)
+       return opcode->flags & F_DELAYED;
+    }
+  return 0;
+}
+
+/* extern void qsort (); */
+
+/* Records current mask of SPARC_OPCODE_ARCH_FOO values, used to pass value
+   to compare_opcodes.  */
+static unsigned int current_arch_mask;
+
+/* Given BFD mach number, return a mask of SPARC_OPCODE_ARCH_FOO values.  */
+
+static int
+compute_arch_mask (unsigned long mach)
+{
+  switch (mach)
+    {
+    case 0 :
+    case bfd_mach_sparc :
+      return (SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_V8)
+              | SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_LEON));
+    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);
+    case bfd_mach_sparc_v8plusc :
+    case bfd_mach_sparc_v9c :
+      return SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_V9C);
+    case bfd_mach_sparc_v8plusd :
+    case bfd_mach_sparc_v9d :
+      return SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_V9D);
+    case bfd_mach_sparc_v8pluse :
+    case bfd_mach_sparc_v9e :
+      return SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_V9E);
+    case bfd_mach_sparc_v8plusv :
+    case bfd_mach_sparc_v9v :
+      return SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_V9V);
+    case bfd_mach_sparc_v8plusm :
+    case bfd_mach_sparc_v9m :
+      return SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_V9M);
+    case bfd_mach_sparc_v8plusm8 :
+    case bfd_mach_sparc_v9m8 :
+      return SPARC_OPCODE_ARCH_MASK (SPARC_OPCODE_ARCH_M8);
+    }
+  abort ();
+}
+
+/* Compare opcodes A and B.  */
+
+static int
+compare_opcodes (const void * a, const void * b)
+{
+  sparc_opcode *op0 = * (sparc_opcode **) a;
+  sparc_opcode *op1 = * (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)
+    {
+      opcodes_error_handler
+       /* 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)
+    {
+      opcodes_error_handler
+       /* xgettext:c-format */
+       (_("internal error: bad sparc-opcode.h: \"%s\", %#.8lx, %#.8lx\n"),
+        op1->name, match1, lose1);
+      op1->lose &= ~op1->match;
+      lose1 = op1->lose;
+    }
+
+  /* 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)
+    {
+      unsigned long int x = 1ul << i;
+      int x0 = (match0 & x) != 0;
+      int x1 = (match1 & x) != 0;
+
+      if (x0 != x1)
+       return x1 - x0;
+    }
+
+  for (i = 0; i < 32; ++i)
+    {
+      unsigned long int x = 1ul << i;
+      int x0 = (lose0 & x) != 0;
+      int x1 = (lose1 & x) != 0;
+
+      if (x0 != x1)
+       return x1 - x0;
+    }
+
+  /* They are functionally equal.  So as long as the opcode table is
+     valid, we can put whichever one first we want, on aesthetic grounds.  */
+
+  /* Our first aesthetic ground is that aliases defer to real insns.  */
+  {
+    int alias_diff = (op0->flags & F_ALIAS) - (op1->flags & F_ALIAS);
+
+    if (alias_diff != 0)
+      /* Put the one that isn't an alias first.  */
+      return alias_diff;
+  }
 
-  for (i = 0; i < NUMOPCODES; ++i)
+  /* Except for aliases, two "identical" instructions had
+     better have the same opcode.  This is a sanity check on the table.  */
+  i = strcmp (op0->name, op1->name);
+  if (i)
     {
-      const struct sparc_opcode *opcode = &sparc_opcodes[i];
-      if ((opcode->match & insn.code) == opcode->match
-         && (opcode->lose & insn.code) == 0)
-       return (opcode->flags & F_DELAYED);
+      if (op0->flags & F_ALIAS)
+       {
+         if (op0->flags & F_PREFERRED)
+           return -1;
+         if (op1->flags & F_PREFERRED)
+           return 1;
+
+         /* If they're both aliases, and neither is marked as preferred,
+            be arbitrary.  */
+         return i;
+       }
+      else
+       opcodes_error_handler
+         /* xgettext:c-format */
+         (_("internal error: bad sparc-opcode.h: \"%s\" == \"%s\"\n"),
+          op0->name, op1->name);
     }
+
+  /* Fewer arguments are preferred.  */
+  {
+    int length_diff = strlen (op0->args) - strlen (op1->args);
+
+    if (length_diff != 0)
+      /* Put the one with fewer arguments first.  */
+      return length_diff;
+  }
+
+  /* Put 1+i before i+1.  */
+  {
+    char *p0 = (char *) strchr (op0->args, '+');
+    char *p1 = (char *) strchr (op1->args, '+');
+
+    if (p0 && p1)
+      {
+       /* There is a plus in both operands.  Note that a plus
+          sign cannot be the first character in args,
+          so the following [-1]'s are valid.  */
+       if (p0[-1] == 'i' && p1[1] == 'i')
+         /* op0 is i+1 and op1 is 1+i, so op1 goes first.  */
+         return 1;
+       if (p0[1] == 'i' && p1[-1] == 'i')
+         /* op0 is 1+i and op1 is i+1, so op0 goes first.  */
+         return -1;
+      }
+  }
+
+  /* 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;
 }
 
-static int opcodes_sorted = 0;
-extern void qsort ();
+/* 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 (const sparc_opcode **opcode_table,
+                 sparc_opcode_hash **hash_table,
+                 int num_opcodes)
+{
+  int i;
+  int hash_count[HASH_SIZE];
+  static sparc_opcode_hash *hash_buf = NULL;
+
+  /* Start at the end of the table and work backwards so that each
+     chain is sorted.  */
+
+  memset (hash_table, 0, HASH_SIZE * sizeof (hash_table[0]));
+  memset (hash_count, 0, HASH_SIZE * sizeof (hash_count[0]));
+  free (hash_buf);
+  hash_buf = xmalloc (sizeof (* hash_buf) * num_opcodes);
+  for (i = num_opcodes - 1; i >= 0; --i)
+    {
+      int hash = HASH_INSN (opcode_table[i]->match);
+      sparc_opcode_hash *h = &hash_buf[i];
+
+      h->next = hash_table[hash];
+      h->opcode = opcode_table[i];
+      hash_table[hash] = h;
+      ++hash_count[hash];
+    }
+
+#if 0 /* for debugging */
+  {
+    int min_count = num_opcodes, max_count = 0;
+    int total;
+
+    for (i = 0; i < HASH_SIZE; ++i)
+      {
+        if (hash_count[i] < min_count)
+         min_count = hash_count[i];
+       if (hash_count[i] > max_count)
+         max_count = hash_count[i];
+       total += hash_count[i];
+      }
+
+    printf ("Opcode hash table stats: min %d, max %d, ave %f\n",
+           min_count, max_count, (double) total / HASH_SIZE);
+  }
+#endif
+}
 
-/* Print one instruction from MEMADDR on STREAM.
+/* Print one instruction from MEMADDR on INFO->STREAM.
 
    We suffix the instruction with a comment that gives the absolute
    address involved, as well as its symbolic form, if the instruction
    is preceded by a findable `sethi' and it either adds an immediate
    displacement to that register, or it is an `add' or `or' instruction
    on that register.  */
+
 int
-print_insn_sparc (memaddr, info)
-     bfd_vma memaddr;
-     disassemble_info *info;
+print_insn_sparc (bfd_vma memaddr, disassemble_info *info)
 {
   FILE *stream = info->stream;
-  union sparc_insn insn;
-
-  register unsigned int i;
-
-  if (!opcodes_sorted)
+  bfd_byte buffer[4];
+  unsigned long insn;
+  sparc_opcode_hash *op;
+  /* 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) (const void *);
+
+  if (!opcodes_initialized
+      || info->mach != current_mach)
     {
-      static int compare_opcodes ();
-      qsort ((char *) sparc_opcodes, NUMOPCODES,
-            sizeof (sparc_opcodes[0]), compare_opcodes);
-      opcodes_sorted = 1;
+      int i;
+
+      current_arch_mask = compute_arch_mask (info->mach);
+
+      if (!opcodes_initialized)
+       sorted_opcodes =
+         xmalloc (sparc_num_opcodes * sizeof (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;
     }
 
   {
     int status =
-      (*info->read_memory_func) (memaddr, (char *) &insn, sizeof (insn), info);
+      (*info->read_memory_func) (memaddr, buffer, sizeof (buffer), info);
+
     if (status != 0)
       {
        (*info->memory_error_func) (status, memaddr, info);
@@ -152,75 +527,103 @@ print_insn_sparc (memaddr, info)
       }
   }
 
-  for (i = 0; i < NUMOPCODES; ++i)
+  /* 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.  */
+  info->branch_delay_insns = 0;                        /* Assume no delay.  */
+  info->target = 0;                            /* Assume no target known.  */
+
+  for (op = opcode_hash_table[HASH_INSN (insn)]; op; op = op->next)
     {
-      const struct sparc_opcode *opcode = &sparc_opcodes[i];
-      if ((opcode->match & insn.code) == opcode->match
-         && (opcode->lose & insn.code) == 0)
+      const 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)
        {
          /* 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.  */
          int found_plus = 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)  */
-             && insn.rs1 == insn.rd)
+
+         /* Nonzero means we have an annulled branch.  */
+         int is_annulled = 0;
+
+         /* 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 (insn.rs1 != insn.rd
+         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);
+         (*info->fprintf_func) (stream, "%s", opcode->name);
 
          {
-           register const char *s;
+           const char *s;
 
            if (opcode->args[0] != ',')
              (*info->fprintf_func) (stream, " ");
+
            for (s = opcode->args; *s != '\0'; ++s)
              {
                while (*s == ',')
                  {
                    (*info->fprintf_func) (stream, ",");
                    ++s;
-                   switch (*s) {
-                   case 'a':
-                     (*info->fprintf_func) (stream, "a");
-                     ++s;
-                     continue;
-#ifndef NO_V9
-                   case 'N':
-                     (*info->fprintf_func) (stream, "pn");
-                     ++s;
-                     continue;
-
-                   case 'T':
-                     (*info->fprintf_func) (stream, "pt");
-                     ++s;
-                     continue;
-#endif                         /* NO_V9 */
-
-                   default:
-                     break;
-                   }           /* switch on arg */
-                 }             /* while there are comma started args */
+                   switch (*s)
+                     {
+                     case 'a':
+                       (*info->fprintf_func) (stream, "a");
+                       is_annulled = 1;
+                       ++s;
+                       continue;
+                     case 'N':
+                       (*info->fprintf_func) (stream, "pn");
+                       ++s;
+                       continue;
+
+                     case 'T':
+                       (*info->fprintf_func) (stream, "pt");
+                       ++s;
+                       continue;
+
+                     default:
+                       break;
+                     }
+                 }
 
                (*info->fprintf_func) (stream, " ");
-                       
+
                switch (*s)
                  {
                  case '+':
                    found_plus = 1;
+                   /* Fall through.  */
 
-                   /* note fall-through */
                  default:
                    (*info->fprintf_func) (stream, "%c", *s);
                    break;
@@ -232,62 +635,97 @@ print_insn_sparc (memaddr, info)
 #define        reg(n)  (*info->fprintf_func) (stream, "%%%s", reg_names[n])
                  case '1':
                  case 'r':
-                   reg (insn.rs1);
+                   reg (X_RS1 (insn));
                    break;
 
                  case '2':
-                   reg (insn.rs2);
+                 case 'O':
+                   reg (X_RS2 (insn));
                    break;
 
                  case 'd':
-                   reg (insn.rd);
+                   reg (X_RD (insn));
                    break;
 #undef reg
 
-#define        freg(n) (*info->fprintf_func) (stream, "%%%s", freg_names[n])
+#define        freg(n)         (*info->fprintf_func) (stream, "%%%s", freg_names[n])
+#define        fregx(n)        (*info->fprintf_func) (stream, "%%%s", freg_names[((n) & ~1) | (((n) & 1) << 5)])
                  case 'e':
-                 case 'v':     /* double/even */
-                 case 'V':     /* quad/multiple of 4 */
-                   freg (insn.rs1);
+                   freg (X_RS1 (insn));
+                   break;
+                 case 'v':     /* Double/even.  */
+                 case 'V':     /* Quad/multiple of 4.  */
+                  case ';':    /* Double/even multiple of 8 doubles.  */
+                   fregx (X_RS1 (insn));
                    break;
 
                  case 'f':
-                 case 'B':     /* double/even */
-                 case 'R':     /* quad/multiple of 4 */
-                   freg (insn.rs2);
+                   freg (X_RS2 (insn));
+                   break;
+                 case 'B':     /* Double/even.  */
+                 case 'R':     /* Quad/multiple of 4.  */
+                  case ':':    /* Double/even multiple of 8 doubles.  */
+                   fregx (X_RS2 (insn));
+                   break;
+
+                 case '4':
+                   freg (X_RS3 (insn));
+                   break;
+                 case '5':     /* Double/even.  */
+                   fregx (X_RS3 (insn));
                    break;
 
                  case 'g':
-                 case 'H':     /* double/even */
-                 case 'J':     /* quad/multiple of 4 */
-                   freg (insn.rd);
+                   freg (X_RD (insn));
+                   break;
+                 case 'H':     /* Double/even.  */
+                 case 'J':     /* Quad/multiple of 4.  */
+                 case '}':     /* Double/even.  */
+                   fregx (X_RD (insn));
                    break;
+                    
+                  case '^':    /* Double/even multiple of 8 doubles.  */
+                    fregx (X_RD (insn) & ~0x6);
+                    break;
+                    
+                  case '\'':   /* Double/even in FPCMPSHL.  */
+                    fregx (X_RS2 (insn | 0x11));
+                    break;
+                    
 #undef freg
+#undef fregx
 
 #define        creg(n) (*info->fprintf_func) (stream, "%%c%u", (unsigned int) (n))
                  case 'b':
-                   creg (insn.rs1);
+                   creg (X_RS1 (insn));
                    break;
 
                  case 'c':
-                   creg (insn.rs2);
+                   creg (X_RS2 (insn));
                    break;
 
                  case 'D':
-                   creg (insn.rd);
+                   creg (X_RD (insn));
                    break;
 #undef creg
 
                  case 'h':
                    (*info->fprintf_func) (stream, "%%hi(%#x)",
-                                          (int) insn.imm22 << 10);
+                                          (unsigned) X_IMM22 (insn) << 10);
                    break;
 
-                 case 'i':
+                 case 'i':     /* 13 bit immediate.  */
+                 case 'I':     /* 11 bit immediate.  */
+                 case 'j':     /* 10 bit immediate.  */
                    {
-                     /* We cannot trust the compiler to sign-extend
-                        when extracting the bitfield, hence the shifts.  */
-                     int imm = ((int) insn.imm13 << 19) >> 19;
+                     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.
@@ -298,7 +736,7 @@ print_insn_sparc (memaddr, info)
                         not before it.  */
                      if (found_plus)
                        imm_added_to_rs1 = 1;
-                     
+
                      if (imm <= 9)
                        (*info->fprintf_func) (stream, "%d", imm);
                      else
@@ -306,29 +744,15 @@ print_insn_sparc (memaddr, info)
                    }
                    break;
 
-#ifndef NO_V9
-                 case 'I':     /* 11 bit immediate.  */
-                 case 'j':     /* 10 bit immediate.  */
-                   {
-                     /* We cannot trust the compiler to sign-extend
-                        when extracting the bitfield, hence the shifts.  */
-                     int imm;
+                 case ')':     /* 5 bit unsigned immediate from RS3.  */
+                   (info->fprintf_func) (stream, "%#x", (unsigned int) X_RS3 (insn));
+                   break;
 
-                     if (*s == 'I')
-                       imm = ((int) insn.imm13 << 21) >> 21;
-                     else
-                       imm = ((int) insn.imm13 << 22) >> 22;
+                 case 'X':     /* 5 bit unsigned immediate.  */
+                 case 'Y':     /* 6 bit unsigned immediate.  */
+                   {
+                     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
@@ -336,112 +760,251 @@ print_insn_sparc (memaddr, info)
                    }
                    break;
 
+                 case '3':
+                   (info->fprintf_func) (stream, "%ld", 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 '=':
+                   info->target = memaddr + SEX (X_DISP10 (insn), 10) * 4;
+                   (*info->print_address_func) (info->target, info);
+                   break;
 
                  case 'k':
-                   print_address ((bfd_vma)
-                                  (memaddr
-                                   + (((int) insn.disp14 << 18) >> 18) * 4),
-                                  stream);
+                   info->target = memaddr + SEX (X_DISP16 (insn), 16) * 4;
+                   (*info->print_address_func) (info->target, info);
                    break;
 
                  case 'G':
-                   print_address ((bfd_vma)
-                                  (memaddr
-                                   /* We use only 19 of the 21 bits.  */
-                                   + (((int) insn.disp21 << 13) >> 13) * 4),
-                                  stream);
+                   info->target = memaddr + SEX (X_DISP19 (insn), 19) * 4;
+                   (*info->print_address_func) (info->target, info);
                    break;
 
                  case '6':
                  case '7':
                  case '8':
                  case '9':
-                   fprintf (stream, "fcc%c", *s - '6' + '0');
+                   (*info->fprintf_func) (stream, "%%fcc%c", *s - '6' + '0');
                    break;
 
                  case 'z':
-                   fputs ("icc", stream);
+                   (*info->fprintf_func) (stream, "%%icc");
                    break;
 
                  case 'Z':
-                   fputs ("xcc", stream);
+                   (*info->fprintf_func) (stream, "%%xcc");
                    break;
 
                  case 'E':
-                   fputs ("%ccr", stream);
+                   (*info->fprintf_func) (stream, "%%ccr");
                    break;
 
                  case 's':
-                   fputs ("%fprs", stream);
+                   (*info->fprintf_func) (stream, "%%fprs");
                    break;
-#endif                         /* NO_V9 */
+
+                 case '{':
+                   (*info->fprintf_func) (stream, "%%mcdper");
+                   break;
+
+                  case '&':
+                    (*info->fprintf_func) (stream, "%%entropy");
+                    break;
+
+                 case 'o':
+                   (*info->fprintf_func) (stream, "%%asi");
+                   break;
+
+                 case 'W':
+                   (*info->fprintf_func) (stream, "%%tick");
+                   break;
+
+                 case 'P':
+                   (*info->fprintf_func) (stream, "%%pc");
+                   break;
+
+                 case '?':
+                   if (X_RS1 (insn) == 31)
+                     (*info->fprintf_func) (stream, "%%ver");
+                   else if (X_RS1 (insn) == 23)
+                     (*info->fprintf_func) (stream, "%%pmcdper");
+                   else if ((unsigned) X_RS1 (insn) < 17)
+                     (*info->fprintf_func) (stream, "%%%s",
+                                            v9_priv_reg_names[X_RS1 (insn)]);
+                   else
+                     (*info->fprintf_func) (stream, "%%reserved");
+                   break;
+
+                 case '!':
+                    if (X_RD (insn) == 31)
+                      (*info->fprintf_func) (stream, "%%ver");
+                   else if (X_RD (insn) == 23)
+                     (*info->fprintf_func) (stream, "%%pmcdper");
+                   else if ((unsigned) X_RD (insn) < 17)
+                     (*info->fprintf_func) (stream, "%%%s",
+                                            v9_priv_reg_names[X_RD (insn)]);
+                   else
+                     (*info->fprintf_func) (stream, "%%reserved");
+                   break;
+
+                 case '$':
+                   if ((unsigned) X_RS1 (insn) < 32)
+                     (*info->fprintf_func) (stream, "%%%s",
+                                            v9_hpriv_reg_names[X_RS1 (insn)]);
+                   else
+                     (*info->fprintf_func) (stream, "%%reserved");
+                   break;
+
+                 case '%':
+                   if ((unsigned) X_RD (insn) < 32)
+                     (*info->fprintf_func) (stream, "%%%s",
+                                            v9_hpriv_reg_names[X_RD (insn)]);
+                   else
+                     (*info->fprintf_func) (stream, "%%reserved");
+                   break;
+
+                 case '/':
+                   if (X_RS1 (insn) < 16 || X_RS1 (insn) > 28)
+                     (*info->fprintf_func) (stream, "%%reserved");
+                   else
+                     (*info->fprintf_func) (stream, "%%%s",
+                                            v9a_asr_reg_names[X_RS1 (insn)-16]);
+                   break;
+
+                 case '_':
+                   if (X_RD (insn) < 16 || X_RD (insn) > 28)
+                     (*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, "%ld", X_RD (insn));
+                     break;
+                   }
 
                  case 'M':
-                   fprintf(stream, "%%asr%d", insn.rs1);
+                   (*info->fprintf_func) (stream, "%%asr%ld", X_RS1 (insn));
                    break;
-                   
+
                  case 'm':
-                   fprintf(stream, "%%asr%d", insn.rd);
+                   (*info->fprintf_func) (stream, "%%asr%ld", X_RD (insn));
                    break;
-                   
+
                  case 'L':
-                   print_address ((bfd_vma) memaddr + insn.disp30 * 4,
-                                  stream);
+                   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));
                    break;
 
                  case 'l':
-                   if ((insn.code >> 22) == 0)
-                     /* Special case for `unimp'.  Don't try to turn
-                        it's operand into a function offset.  */
-                     (*info->fprintf_func)
-                       (stream, "%#x",
-                        (int) (((int) insn.disp22 << 10) >> 10));
-                   else
-                     /* We cannot trust the compiler to sign-extend
-                        when extracting the bitfield, hence the shifts.  */
-                     print_address ((bfd_vma)
-                                    (memaddr
-                                     + (((int) insn.disp22 << 10) >> 10) * 4),
-                                    stream);
+                   info->target = memaddr + SEX (X_DISP22 (insn), 22) * 4;
+                   (*info->print_address_func) (info->target, info);
                    break;
 
                  case 'A':
-                   (*info->fprintf_func) (stream, "(%d)", (int) insn.asi);
-                   break;
+                   {
+                     const char *name = sparc_decode_asi (X_ASI (insn));
+
+                     if (name)
+                       (*info->fprintf_func) (stream, "%s", name);
+                     else
+                       (*info->fprintf_func) (stream, "(%ld)", X_ASI (insn));
+                     break;
+                   }
 
                  case 'C':
-                   (*info->fprintf_func) (stream, "%csr");
+                   (*info->fprintf_func) (stream, "%%csr");
                    break;
 
                  case 'F':
-                   (*info->fprintf_func) (stream, "%fsr");
+                   (*info->fprintf_func) (stream, "%%fsr");
+                   break;
+
+                 case '(':
+                   (*info->fprintf_func) (stream, "%%efsr");
                    break;
 
                  case 'p':
-                   (*info->fprintf_func) (stream, "%psr");
+                   (*info->fprintf_func) (stream, "%%psr");
                    break;
 
                  case 'q':
-                   (*info->fprintf_func) (stream, "%fq");
+                   (*info->fprintf_func) (stream, "%%fq");
                    break;
 
                  case 'Q':
-                   (*info->fprintf_func) (stream, "%cq");
+                   (*info->fprintf_func) (stream, "%%cq");
                    break;
 
                  case 't':
-                   (*info->fprintf_func) (stream, "%tbr");
+                   (*info->fprintf_func) (stream, "%%tbr");
                    break;
 
                  case 'w':
-                   (*info->fprintf_func) (stream, "%wim");
+                   (*info->fprintf_func) (stream, "%%wim");
+                   break;
+
+                 case 'x':
+                   (*info->fprintf_func) (stream, "%ld",
+                                          ((X_LDST_I (insn) << 8)
+                                           + X_ASI (insn)));
                    break;
 
+                  case '|': /* 2-bit immediate  */
+                    (*info->fprintf_func) (stream, "%ld", X_IMM2 (insn));
+                    break;
+
                  case 'y':
-                   (*info->fprintf_func) (stream, "%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;
+                   }
                  }
              }
          }
@@ -452,30 +1015,40 @@ print_insn_sparc (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)
            {
-             union sparc_insn prev_insn;
+             unsigned long prev_insn;
              int errcode;
 
-             errcode =
-               (*info->read_memory_func)
-                 (memaddr - 4,
-                  (char *)&prev_insn, sizeof (prev_insn));
+             if (memaddr >= 4)
+               errcode =
+                 (*info->read_memory_func)
+                 (memaddr - 4, buffer, sizeof (buffer), info);
+             else
+               errcode = 1;
+
+             prev_insn = getword (buffer);
 
              if (errcode == 0)
                {
                  /* If it is a delayed branch, we need to look at the
                     instruction before the delayed branch.  This handles
-                    sequences such as
+                    sequences such as:
 
                     sethi %o1, %hi(_foo), %o1
                     call _printf
-                    or %o1, %lo(_foo), %o1
-                    */
+                    or %o1, %lo(_foo), %o1  */
 
                  if (is_delayed_branch (prev_insn))
-                   errcode = (*info->read_memory_func)
-                     (memaddr - 8, (char *)&prev_insn, sizeof (prev_insn));
+                   {
+                     if (memaddr >= 8)
+                       errcode = (*info->read_memory_func)
+                         (memaddr - 8, buffer, sizeof (buffer), info);
+                     else
+                       errcode = 1;
+
+                     prev_insn = getword (buffer);
+                   }
                }
 
              /* If there was a problem reading memory, then assume
@@ -483,130 +1056,41 @@ print_insn_sparc (memaddr, info)
              if (errcode == 0)
                {
                  /* Is it sethi to the same register?  */
-                 if ((prev_insn.code & 0xc1c00000) == 0x01000000
-                     && prev_insn.rd == insn.rs1)
+                 if ((prev_insn & 0xc1c00000) == 0x01000000
+                     && X_RD (prev_insn) == X_RS1 (insn))
                    {
                      (*info->fprintf_func) (stream, "\t! ");
-                     /* We cannot trust the compiler to sign-extend
-                        when extracting the bitfield, hence the shifts.  */
-                     print_address (((int) prev_insn.imm22 << 10)
-                                    | (insn.imm13 << 19) >> 19, stream);
+                     info->target = (unsigned) 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!!! */
                    }
                }
            }
 
-         return sizeof (insn);
-       }
-    }
-
-  (*info->fprintf_func) (stream, "%#8x", insn.code);
-  return sizeof (insn);
-}
-
-/* Compare opcodes A and B.  */
-
-static int
-compare_opcodes (a, b)
-     char *a, *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 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);
-      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);
-      op1->lose &= ~op1->match;
-      lose1 = op1->lose;
-    }
-
-  /* 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)
-    {
-      unsigned long int x = 1 << i;
-      int x0 = (match0 & x) != 0;
-      int x1 = (match1 & x) != 0;
-
-      if (x0 != x1)
-       return x1 - x0;
-    }
-
-  for (i = 0; i < 32; ++i)
-    {
-      unsigned long int x = 1 << i;
-      int x0 = (lose0 & x) != 0;
-      int x1 = (lose1 & x) != 0;
+         if (opcode->flags & (F_UNBR|F_CONDBR|F_JSR))
+           {
+             /* FIXME -- check is_annulled flag.  */
+             (void) is_annulled;
+             if (opcode->flags & F_UNBR)
+               info->insn_type = dis_branch;
+             if (opcode->flags & F_CONDBR)
+               info->insn_type = dis_condbranch;
+             if (opcode->flags & F_JSR)
+               info->insn_type = dis_jsr;
+             if (opcode->flags & F_DELAYED)
+               info->branch_delay_insns = 1;
+           }
 
-      if (x0 != x1)
-       return x1 - x0;
+         return sizeof (buffer);
+       }
     }
 
-  /* They are functionally equal.  So as long as the opcode table is
-     valid, we can put whichever one first we want, on aesthetic grounds.  */
-
-  /* Our first aesthetic ground is that aliases defer to real insns.  */
-  {
-    int alias_diff = (op0->flags & F_ALIAS) - (op1->flags & F_ALIAS);
-    if (alias_diff != 0)
-      /* Put the one that isn't an alias first.  */
-      return alias_diff;
-  }
-
-  /* Except for aliases, two "identical" instructions had
-     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;
-      else
-         fprintf (stderr,
-                  "Internal error: bad sparc-opcode.h: \"%s\" == \"%s\"\n",
-                  op0->name, op1->name);
-
-  /* Fewer arguments are preferred.  */
-  {
-    int length_diff = strlen (op0->args) - strlen (op1->args);
-    if (length_diff != 0)
-      /* Put the one with fewer arguments first.  */
-      return length_diff;
-  }
-
-  /* Put 1+i before i+1.  */
-  {
-    char *p0 = (char *) strchr(op0->args, '+');
-    char *p1 = (char *) strchr(op1->args, '+');
-
-    if (p0 && p1)
-      {
-       /* There is a plus in both operands.  Note that a plus
-          sign cannot be the first character in args,
-          so the following [-1]'s are valid.  */
-       if (p0[-1] == 'i' && p1[1] == 'i')
-         /* op0 is i+1 and op1 is 1+i, so op1 goes first.  */
-         return 1;
-       if (p0[1] == 'i' && p1[-1] == 'i')
-         /* op0 is 1+i and op1 is i+1, so op0 goes first.  */
-         return -1;
-      }
-  }
-
-  /* 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.  */
-  return 0;
+  info->insn_type = dis_noninsn;       /* Mark as non-valid instruction.  */
+  (*info->fprintf_func) (stream, _("unknown"));
+  return sizeof (buffer);
 }
This page took 0.042448 seconds and 4 git commands to generate.