2004-10-30 Andrew Cagney <cagney@gnu.org>

	* mips-tdep.c (gdb_print_insn_mips): Simplify.
	(make_mips16_addr): Delete.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 97801ca84d47c8a568f141a8b87fb2f7ce454a9f..3527a3ba3ae285910c9704e109a63cf9bc166f16 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2004-10-30  Andrew Cagney  <cagney@gnu.org>
+
+       * mips-tdep.c (gdb_print_insn_mips): Simplify.
+       (make_mips16_addr): Delete.
+       
 2004-10-30  Mark Kettenis  <kettenis@gnu.org>
 
        * i386-sol2-nat.c: New file.

diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c
index 2e26ff6880c41ed082f44a9c05deac1c5d415982..bddf0a37289a2fdeba9c6d0e520f212c77a801db 100644 (file)
--- a/gdb/mips-tdep.c
+++ b/gdb/mips-tdep.c
@@ -205,12 +205,6 @@ is_mips16_addr (CORE_ADDR addr)
   return ((addr) & 1);
 }
 
-static CORE_ADDR
-make_mips16_addr (CORE_ADDR addr)
-{
-  return ((addr) | 1);
-}
-
 static CORE_ADDR
 unmake_mips16_addr (CORE_ADDR addr)
 {
@@ -4842,34 +4836,13 @@ static int
 gdb_print_insn_mips (bfd_vma memaddr, struct disassemble_info *info)
 {
   struct gdbarch_tdep *tdep = gdbarch_tdep (current_gdbarch);
-  mips_extra_func_info_t proc_desc;
 
-  /* Search for the function containing this address.  Set the low bit
-     of the address when searching, in case we were given an even address
-     that is the start of a 16-bit function.  If we didn't do this,
-     the search would fail because the symbol table says the function
-     starts at an odd address, i.e. 1 byte past the given address.  */
-  memaddr = ADDR_BITS_REMOVE (memaddr);
-  proc_desc = non_heuristic_proc_desc (make_mips16_addr (memaddr), NULL);
-
-  /* Make an attempt to determine if this is a 16-bit function.  If
-     the procedure descriptor exists and the address therein is odd,
-     it's definitely a 16-bit function.  Otherwise, we have to just
-     guess that if the address passed in is odd, it's 16-bits.  */
   /* FIXME: cagney/2003-06-26: Is this even necessary?  The
      disassembler needs to be able to locally determine the ISA, and
      not rely on GDB.  Otherwize the stand-alone 'objdump -d' will not
      work.  */
-  if (proc_desc)
-    {
-      if (mips_pc_is_mips16 (PROC_LOW_ADDR (proc_desc)))
-       info->mach = bfd_mach_mips16;
-    }
-  else
-    {
-      if (mips_pc_is_mips16 (memaddr))
-       info->mach = bfd_mach_mips16;
-    }
+  if (mips_pc_is_mips16 (memaddr))
+    info->mach = bfd_mach_mips16;
 
   /* Round down the instruction address to the appropriate boundary.  */
   memaddr &= (info->mach == bfd_mach_mips16 ? ~1 : ~3);