* mn10300-tdep.c (osabi.h): Include.
[deliverable/binutils-gdb.git] / gdb / mn10300-tdep.c
index d6d39b904b826a2abd0e4fc5fb3e5d23055246a8..f7af50a04837619e499bf0f5e2c44a58679d2327 100644 (file)
@@ -1,7 +1,7 @@
 /* Target-dependent code for the Matsushita MN10300 for GDB, the GNU debugger.
 
-   Copyright 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003 Free Software
-   Foundation, Inc.
+   Copyright 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005
+   Free Software Foundation, Inc.
 
    This file is part of GDB.
 
    Boston, MA 02111-1307, USA.  */
 
 #include "defs.h"
-#include "frame.h"
-#include "inferior.h"
-#include "target.h"
-#include "value.h"
-#include "bfd.h"
-#include "gdb_string.h"
-#include "gdbcore.h"
-#include "symfile.h"
-#include "regcache.h"
 #include "arch-utils.h"
+#include "dis-asm.h"
+#include "gdbtypes.h"
+#include "regcache.h"
+#include "gdb_string.h"
 #include "gdb_assert.h"
+#include "gdbcore.h"   /* for write_memory_unsigned_integer */
+#include "value.h"
+#include "gdbtypes.h"
+#include "frame.h"
+#include "frame-unwind.h"
+#include "frame-base.h"
+#include "trad-frame.h"
+#include "symtab.h"
+#include "dwarf2-frame.h"
+#include "regcache.h"
+#include "osabi.h"
 
-#define D0_REGNUM 0
-#define D2_REGNUM 2
-#define D3_REGNUM 3
-#define A0_REGNUM 4
-#define A2_REGNUM 6
-#define A3_REGNUM 7
-#define MDR_REGNUM 10
-#define PSW_REGNUM 11
-#define LIR_REGNUM 12
-#define LAR_REGNUM 13
-#define MDRQ_REGNUM 14
-#define E0_REGNUM 15
-#define MCRH_REGNUM 26
-#define MCRL_REGNUM 27
-#define MCVF_REGNUM 28
-
-enum movm_register_bits {
-  movm_exother_bit = 0x01,
-  movm_exreg1_bit  = 0x02,
-  movm_exreg0_bit  = 0x04,
-  movm_other_bit   = 0x08,
-  movm_a3_bit      = 0x10,
-  movm_a2_bit      = 0x20,
-  movm_d3_bit      = 0x40,
-  movm_d2_bit      = 0x80
-};
+#include "mn10300-tdep.h"
+
+/* Forward decl.  */
+extern struct trad_frame_cache *mn10300_frame_unwind_cache (struct frame_info*,
+                                                           void **);
 
-extern void _initialize_mn10300_tdep (void);
-static CORE_ADDR mn10300_analyze_prologue (struct frame_info *fi,
-                                          CORE_ADDR pc);
+/* Compute the alignment required by a type.  */
 
-/* mn10300 private data */
-struct gdbarch_tdep
+static int
+mn10300_type_align (struct type *type)
 {
-  int am33_mode;
-#define AM33_MODE (gdbarch_tdep (current_gdbarch)->am33_mode)
-};
+  int i, align = 1;
 
-/* Additional info used by the frame */
+  switch (TYPE_CODE (type))
+    {
+    case TYPE_CODE_INT:
+    case TYPE_CODE_ENUM:
+    case TYPE_CODE_SET:
+    case TYPE_CODE_RANGE:
+    case TYPE_CODE_CHAR:
+    case TYPE_CODE_BOOL:
+    case TYPE_CODE_FLT:
+    case TYPE_CODE_PTR:
+    case TYPE_CODE_REF:
+      return TYPE_LENGTH (type);
+
+    case TYPE_CODE_COMPLEX:
+      return TYPE_LENGTH (type) / 2;
+
+    case TYPE_CODE_STRUCT:
+    case TYPE_CODE_UNION:
+      for (i = 0; i < TYPE_NFIELDS (type); i++)
+       {
+         int falign = mn10300_type_align (TYPE_FIELD_TYPE (type, i));
+         while (align < falign)
+           align <<= 1;
+       }
+      return align;
 
-struct frame_extra_info
-  {
-    int status;
-    int stack_size;
-  };
+    case TYPE_CODE_ARRAY:
+      /* HACK!  Structures containing arrays, even small ones, are not
+        elligible for returning in registers.  */
+      return 256;
 
+    case TYPE_CODE_TYPEDEF:
+      return mn10300_type_align (check_typedef (type));
+
+    default:
+      internal_error (__FILE__, __LINE__, _("bad switch"));
+    }
+}
+
+/* MVS note this is deprecated.  */
+/* Should call_function allocate stack space for a struct return?  */
+/* gcc_p unused */
+static int
+mn10300_use_struct_convention (int gcc_p, struct type *type)
+{
+  /* Structures bigger than a pair of words can't be returned in
+     registers.  */
+  if (TYPE_LENGTH (type) > 8)
+    return 1;
+
+  switch (TYPE_CODE (type))
+    {
+    case TYPE_CODE_STRUCT:
+    case TYPE_CODE_UNION:
+      /* Structures with a single field are handled as the field
+        itself.  */
+      if (TYPE_NFIELDS (type) == 1)
+       return mn10300_use_struct_convention (gcc_p, 
+                                             TYPE_FIELD_TYPE (type, 0));
+
+      /* Structures with word or double-word size are passed in memory, as
+        long as they require at least word alignment.  */
+      if (mn10300_type_align (type) >= 4)
+       return 0;
+
+      return 1;
+
+      /* Arrays are addressable, so they're never returned in
+        registers.  This condition can only hold when the array is
+        the only field of a struct or union.  */
+    case TYPE_CODE_ARRAY:
+      return 1;
+
+    case TYPE_CODE_TYPEDEF:
+      return mn10300_use_struct_convention (gcc_p, check_typedef (type));
+
+    default:
+      return 0;
+    }
+}
+
+/* MVS note this is deprecated.  */
+static void
+mn10300_store_return_value (struct type *type,
+                           struct regcache *regcache, const void *valbuf)
+{
+  struct gdbarch *gdbarch = get_regcache_arch (regcache);
+  int len = TYPE_LENGTH (type);
+  int reg, regsz;
+  
+  if (TYPE_CODE (type) == TYPE_CODE_PTR)
+    reg = 4;
+  else
+    reg = 0;
+
+  regsz = register_size (gdbarch, reg);
+
+  if (len <= regsz)
+    regcache_raw_write_part (regcache, reg, 0, len, valbuf);
+  else if (len <= 2 * regsz)
+    {
+      regcache_raw_write (regcache, reg, valbuf);
+      gdb_assert (regsz == register_size (gdbarch, reg + 1));
+      regcache_raw_write_part (regcache, reg+1, 0,
+                              len - regsz, (char *) valbuf + regsz);
+    }
+  else
+    internal_error (__FILE__, __LINE__,
+                   _("Cannot store return value %d bytes long."), len);
+}
+
+/* MVS note deprecated.  */
+static void
+mn10300_extract_return_value (struct type *type,
+                             struct regcache *regcache, void *valbuf)
+{
+  struct gdbarch *gdbarch = get_regcache_arch (regcache);
+  char buf[MAX_REGISTER_SIZE];
+  int len = TYPE_LENGTH (type);
+  int reg, regsz;
+
+  if (TYPE_CODE (type) == TYPE_CODE_PTR)
+    reg = 4;
+  else
+    reg = 0;
+
+  regsz = register_size (gdbarch, reg);
+  if (len <= regsz)
+    {
+      regcache_raw_read (regcache, reg, buf);
+      memcpy (valbuf, buf, len);
+    }
+  else if (len <= 2 * regsz)
+    {
+      regcache_raw_read (regcache, reg, buf);
+      memcpy (valbuf, buf, regsz);
+      gdb_assert (regsz == register_size (gdbarch, reg + 1));
+      regcache_raw_read (regcache, reg + 1, buf);
+      memcpy ((char *) valbuf + regsz, buf, len - regsz);
+    }
+  else
+    internal_error (__FILE__, __LINE__,
+                   _("Cannot extract return value %d bytes long."), len);
+}
 
 static char *
 register_name (int reg, char **regs, long sizeof_regs)
@@ -113,78 +230,24 @@ am33_register_name (int reg)
   };
   return register_name (reg, regs, sizeof regs);
 }
-  
-static CORE_ADDR
-mn10300_saved_pc_after_call (struct frame_info *fi)
-{
-  return read_memory_integer (read_register (SP_REGNUM), 4);
-}
 
-static void
-mn10300_extract_return_value (struct type *type, char *regbuf, char *valbuf)
+
+static struct type *
+mn10300_register_type (struct gdbarch *gdbarch, int reg)
 {
-  if (TYPE_CODE (type) == TYPE_CODE_PTR)
-    memcpy (valbuf, regbuf + REGISTER_BYTE (4), TYPE_LENGTH (type));
-  else
-    memcpy (valbuf, regbuf + REGISTER_BYTE (0), TYPE_LENGTH (type));
+  return builtin_type_int;
 }
 
 static CORE_ADDR
-mn10300_extract_struct_value_address (char *regbuf)
+mn10300_read_pc (ptid_t ptid)
 {
-  return extract_unsigned_integer (regbuf + REGISTER_BYTE (4),
-                                  REGISTER_RAW_SIZE (4));
+  return read_register_pid (E_PC_REGNUM, ptid);
 }
 
 static void
-mn10300_store_return_value (struct type *type, char *valbuf)
-{
-  if (TYPE_CODE (type) == TYPE_CODE_PTR)
-    deprecated_write_register_bytes (REGISTER_BYTE (4), valbuf,
-                                    TYPE_LENGTH (type));
-  else
-    deprecated_write_register_bytes (REGISTER_BYTE (0), valbuf,
-                                    TYPE_LENGTH (type));
-}
-
-static struct frame_info *analyze_dummy_frame (CORE_ADDR, CORE_ADDR);
-static struct frame_info *
-analyze_dummy_frame (CORE_ADDR pc, CORE_ADDR frame)
-{
-  static struct frame_info *dummy = NULL;
-  if (dummy == NULL)
-    {
-      struct frame_extra_info *extra_info;
-      CORE_ADDR *saved_regs;
-      dummy = deprecated_frame_xmalloc ();
-      saved_regs = xmalloc (SIZEOF_FRAME_SAVED_REGS);
-      deprecated_set_frame_saved_regs_hack (dummy, saved_regs);
-      extra_info = XMALLOC (struct frame_extra_info);
-      deprecated_set_frame_extra_info_hack (dummy, extra_info);
-    }
-  deprecated_set_frame_next_hack (dummy, NULL);
-  deprecated_set_frame_prev_hack (dummy, NULL);
-  deprecated_update_frame_pc_hack (dummy, pc);
-  deprecated_update_frame_base_hack (dummy, frame);
-  get_frame_extra_info (dummy)->status = 0;
-  get_frame_extra_info (dummy)->stack_size = 0;
-  memset (get_frame_saved_regs (dummy), '\000', SIZEOF_FRAME_SAVED_REGS);
-  mn10300_analyze_prologue (dummy, 0);
-  return dummy;
-}
-
-/* Values for frame_info.status */
-
-#define MY_FRAME_IN_SP 0x1
-#define MY_FRAME_IN_FP 0x2
-#define NO_MORE_FRAMES 0x4
-
-
-/* Should call_function allocate stack space for a struct return?  */
-static int
-mn10300_use_struct_convention (int gcc_p, struct type *type)
+mn10300_write_pc (CORE_ADDR val, ptid_t ptid)
 {
-  return (TYPE_NFIELDS (type) > 1 || TYPE_LENGTH (type) > 8);
+  return write_register_pid (E_PC_REGNUM, val, ptid);
 }
 
 /* The breakpoint instruction must be the same size as the smallest
@@ -197,26 +260,67 @@ mn10300_use_struct_convention (int gcc_p, struct type *type)
 const static unsigned char *
 mn10300_breakpoint_from_pc (CORE_ADDR *bp_addr, int *bp_size)
 {
-  static char breakpoint[] =
-  {0xff};
+  static char breakpoint[] = {0xff};
   *bp_size = 1;
   return breakpoint;
 }
 
+/* 
+ * Frame Extra Info:
+ *
+ *   status -- actually frame type (SP, FP, or last frame)
+ *   stack size -- offset to the next frame
+ * 
+ * The former might ultimately be stored in the frame_base.
+ * Seems like there'd be a way to store the later too.
+ *
+ * Temporarily supply empty stub functions as place holders.
+ */
 
-/* Fix fi->frame if it's bogus at this point.  This is a helper
-   function for mn10300_analyze_prologue. */
+static void
+my_frame_is_in_sp (struct frame_info *fi, void **this_cache)
+{
+  struct trad_frame_cache *cache = mn10300_frame_unwind_cache (fi, this_cache);
+  trad_frame_set_this_base (cache, 
+                           frame_unwind_register_unsigned (fi, 
+                                                           E_SP_REGNUM));
+}
 
 static void
-fix_frame_pointer (struct frame_info *fi, int stack_size)
+my_frame_is_in_fp (struct frame_info *fi, void **this_cache)
+{
+  struct trad_frame_cache *cache = mn10300_frame_unwind_cache (fi, this_cache);
+  trad_frame_set_this_base (cache, 
+                           frame_unwind_register_unsigned (fi, 
+                                                           E_A3_REGNUM));
+}
+
+static void
+my_frame_is_last (struct frame_info *fi)
+{
+}
+
+static int
+is_my_frame_in_sp (struct frame_info *fi)
+{
+  return 0;
+}
+
+static int
+is_my_frame_in_fp (struct frame_info *fi)
+{
+  return 0;
+}
+
+static int
+is_my_frame_last (struct frame_info *fi)
+{
+  return 0;
+}
+
+static void
+set_my_stack_size (struct frame_info *fi, CORE_ADDR size)
 {
-  if (fi && get_next_frame (fi) == NULL)
-    {
-      if (get_frame_extra_info (fi)->status & MY_FRAME_IN_SP)
-       deprecated_update_frame_base_hack (fi, read_sp () - stack_size);
-      else if (get_frame_extra_info (fi)->status & MY_FRAME_IN_FP)
-       deprecated_update_frame_base_hack (fi, read_register (A3_REGNUM));
-    }
 }
 
 
@@ -224,77 +328,91 @@ fix_frame_pointer (struct frame_info *fi, int stack_size)
    This is a helper function for mn10300_analyze_prologue.  */
 
 static void
-set_movm_offsets (struct frame_info *fi, int movm_args)
+set_movm_offsets (struct frame_info *fi, 
+                 void **this_cache, 
+                 int movm_args)
 {
+  struct trad_frame_cache *cache;
   int offset = 0;
+  CORE_ADDR base;
+
+  if (fi == NULL || this_cache == NULL)
+    return;
 
-  if (fi == NULL || movm_args == 0)
+  cache = mn10300_frame_unwind_cache (fi, this_cache);
+  if (cache == NULL)
     return;
 
+  base = trad_frame_get_this_base (cache);
   if (movm_args & movm_other_bit)
     {
       /* The `other' bit leaves a blank area of four bytes at the
          beginning of its block of saved registers, making it 32 bytes
          long in total.  */
-      get_frame_saved_regs (fi)[LAR_REGNUM]    = get_frame_base (fi) + offset + 4;
-      get_frame_saved_regs (fi)[LIR_REGNUM]    = get_frame_base (fi) + offset + 8;
-      get_frame_saved_regs (fi)[MDR_REGNUM]    = get_frame_base (fi) + offset + 12;
-      get_frame_saved_regs (fi)[A0_REGNUM + 1] = get_frame_base (fi) + offset + 16;
-      get_frame_saved_regs (fi)[A0_REGNUM]     = get_frame_base (fi) + offset + 20;
-      get_frame_saved_regs (fi)[D0_REGNUM + 1] = get_frame_base (fi) + offset + 24;
-      get_frame_saved_regs (fi)[D0_REGNUM]     = get_frame_base (fi) + offset + 28;
+      trad_frame_set_reg_addr (cache, E_LAR_REGNUM,    base + offset + 4);
+      trad_frame_set_reg_addr (cache, E_LIR_REGNUM,    base + offset + 8);
+      trad_frame_set_reg_addr (cache, E_MDR_REGNUM,    base + offset + 12);
+      trad_frame_set_reg_addr (cache, E_A0_REGNUM + 1, base + offset + 16);
+      trad_frame_set_reg_addr (cache, E_A0_REGNUM,     base + offset + 20);
+      trad_frame_set_reg_addr (cache, E_D0_REGNUM + 1, base + offset + 24);
+      trad_frame_set_reg_addr (cache, E_D0_REGNUM,     base + offset + 28);
       offset += 32;
     }
+
   if (movm_args & movm_a3_bit)
     {
-      get_frame_saved_regs (fi)[A3_REGNUM] = get_frame_base (fi) + offset;
+      trad_frame_set_reg_addr (cache, E_A3_REGNUM, base + offset);
       offset += 4;
     }
   if (movm_args & movm_a2_bit)
     {
-      get_frame_saved_regs (fi)[A2_REGNUM] = get_frame_base (fi) + offset;
+      trad_frame_set_reg_addr (cache, E_A2_REGNUM, base + offset);
       offset += 4;
     }
   if (movm_args & movm_d3_bit)
     {
-      get_frame_saved_regs (fi)[D3_REGNUM] = get_frame_base (fi) + offset;
+      trad_frame_set_reg_addr (cache, E_D3_REGNUM, base + offset);
       offset += 4;
     }
   if (movm_args & movm_d2_bit)
     {
-      get_frame_saved_regs (fi)[D2_REGNUM] = get_frame_base (fi) + offset;
+      trad_frame_set_reg_addr (cache, E_D2_REGNUM, base + offset);
       offset += 4;
     }
   if (AM33_MODE)
     {
       if (movm_args & movm_exother_bit)
         {
-          get_frame_saved_regs (fi)[MCVF_REGNUM]   = get_frame_base (fi) + offset;
-          get_frame_saved_regs (fi)[MCRL_REGNUM]   = get_frame_base (fi) + offset + 4;
-          get_frame_saved_regs (fi)[MCRH_REGNUM]   = get_frame_base (fi) + offset + 8;
-          get_frame_saved_regs (fi)[MDRQ_REGNUM]   = get_frame_base (fi) + offset + 12;
-          get_frame_saved_regs (fi)[E0_REGNUM + 1] = get_frame_base (fi) + offset + 16;
-          get_frame_saved_regs (fi)[E0_REGNUM + 0] = get_frame_base (fi) + offset + 20;
+         trad_frame_set_reg_addr (cache, E_MCVF_REGNUM, base + offset);
+         trad_frame_set_reg_addr (cache, E_MCRL_REGNUM, base + offset + 4);
+         trad_frame_set_reg_addr (cache, E_MCRH_REGNUM, base + offset + 8);
+         trad_frame_set_reg_addr (cache, E_MDRQ_REGNUM, base + offset + 12);
+         trad_frame_set_reg_addr (cache, E_E1_REGNUM,   base + offset + 16);
+         trad_frame_set_reg_addr (cache, E_E0_REGNUM,   base + offset + 20);
           offset += 24;
         }
       if (movm_args & movm_exreg1_bit)
         {
-          get_frame_saved_regs (fi)[E0_REGNUM + 7] = get_frame_base (fi) + offset;
-          get_frame_saved_regs (fi)[E0_REGNUM + 6] = get_frame_base (fi) + offset + 4;
-          get_frame_saved_regs (fi)[E0_REGNUM + 5] = get_frame_base (fi) + offset + 8;
-          get_frame_saved_regs (fi)[E0_REGNUM + 4] = get_frame_base (fi) + offset + 12;
+         trad_frame_set_reg_addr (cache, E_E7_REGNUM, base + offset);
+         trad_frame_set_reg_addr (cache, E_E6_REGNUM, base + offset + 4);
+         trad_frame_set_reg_addr (cache, E_E5_REGNUM, base + offset + 8);
+         trad_frame_set_reg_addr (cache, E_E4_REGNUM, base + offset + 12);
           offset += 16;
         }
       if (movm_args & movm_exreg0_bit)
         {
-          get_frame_saved_regs (fi)[E0_REGNUM + 3] = get_frame_base (fi) + offset;
-          get_frame_saved_regs (fi)[E0_REGNUM + 2] = get_frame_base (fi) + offset + 4;
+         trad_frame_set_reg_addr (cache, E_E3_REGNUM, base + offset);
+         trad_frame_set_reg_addr (cache, E_E2_REGNUM, base + offset + 4);
           offset += 8;
         }
     }
+  /* The last (or first) thing on the stack will be the PC.  */
+  trad_frame_set_reg_addr (cache, E_PC_REGNUM, base + offset);
+  /* Save the SP in the 'traditional' way.  
+     This will be the same location where the PC is saved.  */
+  trad_frame_set_reg_value (cache, E_SP_REGNUM, base + offset);
 }
 
-
 /* The main purpose of this file is dealing with prologues to extract
    information about stack frames and saved registers.
 
@@ -391,24 +509,39 @@ set_movm_offsets (struct frame_info *fi, int movm_args)
    frame chain to not bother trying to unwind past this frame.  */
 
 static CORE_ADDR
-mn10300_analyze_prologue (struct frame_info *fi, CORE_ADDR pc)
+mn10300_analyze_prologue (struct frame_info *fi, 
+                         void **this_cache, 
+                         CORE_ADDR pc)
 {
   CORE_ADDR func_addr, func_end, addr, stop;
-  CORE_ADDR stack_size;
+  long      stack_size;
   int imm_size;
   unsigned char buf[4];
   int status, movm_args = 0;
   char *name;
 
   /* Use the PC in the frame if it's provided to look up the
-     start of this function.  */
-  pc = (fi ? get_frame_pc (fi) : pc);
+     start of this function.
+
+     Note: kevinb/2003-07-16: We used to do the following here:
+       pc = (fi ? get_frame_pc (fi) : pc);
+     But this is (now) badly broken when called from analyze_dummy_frame().
+  */
+  if (fi)
+    {
+      pc = (pc ? pc : get_frame_pc (fi));
+      /* At the start of a function our frame is in the stack pointer.  */
+      my_frame_is_in_sp (fi, this_cache);
+    }
 
   /* Find the start of this function.  */
   status = find_pc_partial_function (pc, &name, &func_addr, &func_end);
 
-  /* Do nothing if we couldn't find the start of this function or if we're
-     stopped at the first instruction in the prologue.  */
+  /* Do nothing if we couldn't find the start of this function 
+
+     MVS: comment went on to say "or if we're stopped at the first
+     instruction in the prologue" -- but code doesn't reflect that, 
+     and I don't want to do that anyway.  */
   if (status == 0)
     {
       return pc;
@@ -418,20 +551,19 @@ mn10300_analyze_prologue (struct frame_info *fi, CORE_ADDR pc)
   if (strcmp (name, "start") == 0)
     {
       if (fi != NULL)
-       get_frame_extra_info (fi)->status = NO_MORE_FRAMES;
+       my_frame_is_last (fi);
       return pc;
     }
 
-  /* At the start of a function our frame is in the stack pointer.  */
-  if (fi)
-    get_frame_extra_info (fi)->status = MY_FRAME_IN_SP;
-
+#if 0
   /* Get the next two bytes into buf, we need two because rets is a two
      byte insn and the first isn't enough to uniquely identify it.  */
-  status = read_memory_nobpt (pc, buf, 2);
+  status = deprecated_read_memory_nobpt (pc, buf, 2);
   if (status != 0)
     return pc;
 
+  /* Note: kevinb/2003-07-16: We shouldn't be making these sorts of
+     changes to the frame in prologue examination code.  */
   /* If we're physically on an "rets" instruction, then our frame has
      already been deallocated.  Note this can also be true for retf
      and ret if they specify a size of zero.
@@ -452,9 +584,14 @@ mn10300_analyze_prologue (struct frame_info *fi, CORE_ADDR pc)
        deprecated_update_frame_base_hack (fi, read_sp ());
       return get_frame_pc (fi);
     }
+#endif
+
+  /* NOTE: from here on, we don't want to return without jumping to
+     finish_prologue.  */
+
 
   /* Figure out where to stop scanning.  */
-  stop = fi ? get_frame_pc (fi) : func_end;
+  stop = fi ? pc : func_end;
 
   /* Don't walk off the end of the function.  */
   stop = stop > func_end ? func_end : stop;
@@ -463,12 +600,9 @@ mn10300_analyze_prologue (struct frame_info *fi, CORE_ADDR pc)
   addr = func_addr;
 
   /* Suck in two bytes.  */
-  status = read_memory_nobpt (addr, buf, 2);
-  if (status != 0)
-    {
-      fix_frame_pointer (fi, 0);
-      return addr;
-    }
+  if (addr + 2 >= stop
+      || (status = deprecated_read_memory_nobpt (addr, buf, 2)) != 0)
+    goto finish_prologue;
 
   /* First see if this insn sets the stack pointer from a register; if
      so, it's probably the initialization of the stack pointer in _start,
@@ -476,8 +610,8 @@ mn10300_analyze_prologue (struct frame_info *fi, CORE_ADDR pc)
   if (buf[0] == 0xf2 && (buf[1] & 0xf3) == 0xf0)
     {
       if (fi)
-       get_frame_extra_info (fi)->status = NO_MORE_FRAMES;
-      return addr;
+       my_frame_is_last (fi);
+      goto finish_prologue;
     }
 
   /* Now look for movm [regs],sp, which saves the callee saved registers.
@@ -488,35 +622,18 @@ mn10300_analyze_prologue (struct frame_info *fi, CORE_ADDR pc)
   if (buf[0] == 0xcf)
     {
       /* Extract the register list for the movm instruction.  */
-      status = read_memory_nobpt (addr + 1, buf, 1);
-      movm_args = *buf;
+      movm_args = buf[1];
 
       addr += 2;
 
       /* Quit now if we're beyond the stop point.  */
       if (addr >= stop)
-       {
-         /* Fix fi->frame since it's bogus at this point.  */
-         if (fi && get_next_frame (fi) == NULL)
-           deprecated_update_frame_base_hack (fi, read_sp ());
-
-         /* Note if/where callee saved registers were saved.  */
-         set_movm_offsets (fi, movm_args);
-         return addr;
-       }
+       goto finish_prologue;
 
       /* Get the next two bytes so the prologue scan can continue.  */
-      status = read_memory_nobpt (addr, buf, 2);
+      status = deprecated_read_memory_nobpt (addr, buf, 2);
       if (status != 0)
-       {
-         /* Fix fi->frame since it's bogus at this point.  */
-         if (fi && get_next_frame (fi) == NULL)
-           deprecated_update_frame_base_hack (fi, read_sp ());
-
-         /* Note if/where callee saved registers were saved.  */
-         set_movm_offsets (fi, movm_args);
-         return addr;
-       }
+       goto finish_prologue;
     }
 
   /* Now see if we set up a frame pointer via "mov sp,a3" */
@@ -527,32 +644,17 @@ mn10300_analyze_prologue (struct frame_info *fi, CORE_ADDR pc)
       /* The frame pointer is now valid.  */
       if (fi)
        {
-         get_frame_extra_info (fi)->status |= MY_FRAME_IN_FP;
-         get_frame_extra_info (fi)->status &= ~MY_FRAME_IN_SP;
+         my_frame_is_in_fp (fi, this_cache);
        }
 
       /* Quit now if we're beyond the stop point.  */
       if (addr >= stop)
-       {
-         /* Fix fi->frame if it's bogus at this point.  */
-         fix_frame_pointer (fi, 0);
-
-         /* Note if/where callee saved registers were saved.  */
-         set_movm_offsets (fi, movm_args);
-         return addr;
-       }
+       goto finish_prologue;
 
       /* Get two more bytes so scanning can continue.  */
-      status = read_memory_nobpt (addr, buf, 2);
+      status = deprecated_read_memory_nobpt (addr, buf, 2);
       if (status != 0)
-       {
-         /* Fix fi->frame if it's bogus at this point.  */
-         fix_frame_pointer (fi, 0);
-
-         /* Note if/where callee saved registers were saved.  */
-         set_movm_offsets (fi, movm_args);
-         return addr;
-       }
+       goto finish_prologue;
     }
 
   /* Next we should allocate the local frame.  No more prologue insns
@@ -565,17 +667,6 @@ mn10300_analyze_prologue (struct frame_info *fi, CORE_ADDR pc)
      If none of the above was found, then this prologue has no 
      additional stack.  */
 
-  status = read_memory_nobpt (addr, buf, 2);
-  if (status != 0)
-    {
-      /* Fix fi->frame if it's bogus at this point.  */
-      fix_frame_pointer (fi, 0);
-
-      /* Note if/where callee saved registers were saved.  */
-      set_movm_offsets (fi, movm_args);
-      return addr;
-    }
-
   imm_size = 0;
   if (buf[0] == 0xf8 && buf[1] == 0xfe)
     imm_size = 1;
@@ -588,193 +679,186 @@ mn10300_analyze_prologue (struct frame_info *fi, CORE_ADDR pc)
     {
       /* Suck in imm_size more bytes, they'll hold the size of the
          current frame.  */
-      status = read_memory_nobpt (addr + 2, buf, imm_size);
+      status = deprecated_read_memory_nobpt (addr + 2, buf, imm_size);
       if (status != 0)
-       {
-         /* Fix fi->frame if it's bogus at this point.  */
-         fix_frame_pointer (fi, 0);
-
-         /* Note if/where callee saved registers were saved.  */
-         set_movm_offsets (fi, movm_args);
-         return addr;
-       }
+       goto finish_prologue;
 
       /* Note the size of the stack in the frame info structure.  */
       stack_size = extract_signed_integer (buf, imm_size);
       if (fi)
-       get_frame_extra_info (fi)->stack_size = stack_size;
+       set_my_stack_size (fi, stack_size);
 
       /* We just consumed 2 + imm_size bytes.  */
       addr += 2 + imm_size;
 
       /* No more prologue insns follow, so begin preparation to return.  */
-      /* Fix fi->frame if it's bogus at this point.  */
-      fix_frame_pointer (fi, stack_size);
-
-      /* Note if/where callee saved registers were saved.  */
-      set_movm_offsets (fi, movm_args);
-      return addr;
+      goto finish_prologue;
     }
-
-  /* We never found an insn which allocates local stack space, regardless
-     this is the end of the prologue.  */
-  /* Fix fi->frame if it's bogus at this point.  */
-  fix_frame_pointer (fi, 0);
-
+  /* Do the essentials and get out of here.  */
+ finish_prologue:
   /* Note if/where callee saved registers were saved.  */
-  set_movm_offsets (fi, movm_args);
+  if (fi)
+    set_movm_offsets (fi, this_cache, movm_args);
   return addr;
 }
 
+/* Function: skip_prologue
+   Return the address of the first inst past the prologue of the function.  */
 
-/* Function: saved_regs_size
-   Return the size in bytes of the register save area, based on the
-   saved_regs array in FI.  */
-static int
-saved_regs_size (struct frame_info *fi)
+static CORE_ADDR
+mn10300_skip_prologue (CORE_ADDR pc)
 {
-  int adjust = 0;
-  int i;
-
-  /* Reserve four bytes for every register saved.  */
-  for (i = 0; i < NUM_REGS; i++)
-    if (get_frame_saved_regs (fi)[i])
-      adjust += 4;
-
-  /* If we saved LIR, then it's most likely we used a `movm'
-     instruction with the `other' bit set, in which case the SP is
-     decremented by an extra four bytes, "to simplify calculation
-     of the transfer area", according to the processor manual.  */
-  if (get_frame_saved_regs (fi)[LIR_REGNUM])
-    adjust += 4;
-
-  return adjust;
+  return mn10300_analyze_prologue (NULL, NULL, pc);
 }
 
-
-/* Function: frame_chain
-   Figure out and return the caller's frame pointer given current
-   frame_info struct.
-
-   We don't handle dummy frames yet but we would probably just return the
-   stack pointer that was in use at the time the function call was made?  */
-
-static CORE_ADDR
-mn10300_frame_chain (struct frame_info *fi)
+/* Simple frame_unwind_cache.  
+   This finds the "extra info" for the frame.  */
+struct trad_frame_cache *
+mn10300_frame_unwind_cache (struct frame_info *next_frame,
+                           void **this_prologue_cache)
 {
-  struct frame_info *dummy;
-  /* Walk through the prologue to determine the stack size,
-     location of saved registers, end of the prologue, etc.  */
-  if (get_frame_extra_info (fi)->status == 0)
-    mn10300_analyze_prologue (fi, (CORE_ADDR) 0);
-
-  /* Quit now if mn10300_analyze_prologue set NO_MORE_FRAMES.  */
-  if (get_frame_extra_info (fi)->status & NO_MORE_FRAMES)
-    return 0;
-
-  /* Now that we've analyzed our prologue, determine the frame
-     pointer for our caller.
-
-     If our caller has a frame pointer, then we need to
-     find the entry value of $a3 to our function.
-
-     If fsr.regs[A3_REGNUM] is nonzero, then it's at the memory
-     location pointed to by fsr.regs[A3_REGNUM].
+  struct trad_frame_cache *cache;
+  CORE_ADDR pc, start, end;
+
+  if (*this_prologue_cache)
+    return (*this_prologue_cache);
+
+  cache = trad_frame_cache_zalloc (next_frame);
+  pc = gdbarch_unwind_pc (current_gdbarch, next_frame);
+  mn10300_analyze_prologue (next_frame, (void **) &cache, pc);
+  if (find_pc_partial_function (pc, NULL, &start, &end))
+    trad_frame_set_id (cache, 
+                      frame_id_build (trad_frame_get_this_base (cache), 
+                                      start));
+  else
+    trad_frame_set_id (cache, 
+                      frame_id_build (trad_frame_get_this_base (cache), 
+                                      frame_func_unwind (next_frame)));
 
-     Else it's still in $a3.
+  (*this_prologue_cache) = cache;
+  return cache;
+}
 
-     If our caller does not have a frame pointer, then his
-     frame base is fi->frame + -caller's stack size.  */
+/* Here is a dummy implementation.  */
+static struct frame_id
+mn10300_unwind_dummy_id (struct gdbarch *gdbarch,
+                        struct frame_info *next_frame)
+{
+  return frame_id_build (frame_sp_unwind (next_frame), 
+                        frame_pc_unwind (next_frame));
+}
 
-  /* The easiest way to get that info is to analyze our caller's frame.
-     So we set up a dummy frame and call mn10300_analyze_prologue to
-     find stuff for us.  */
-  dummy = analyze_dummy_frame (DEPRECATED_FRAME_SAVED_PC (fi), get_frame_base (fi));
+/* Trad frame implementation.  */
+static void
+mn10300_frame_this_id (struct frame_info *next_frame,
+                      void **this_prologue_cache,
+                      struct frame_id *this_id)
+{
+  struct trad_frame_cache *cache = 
+    mn10300_frame_unwind_cache (next_frame, this_prologue_cache);
 
-  if (get_frame_extra_info (dummy)->status & MY_FRAME_IN_FP)
-    {
-      /* Our caller has a frame pointer.  So find the frame in $a3 or
-         in the stack.  */
-      if (get_frame_saved_regs (fi)[A3_REGNUM])
-       return (read_memory_integer (get_frame_saved_regs (fi)[A3_REGNUM],
-                                    DEPRECATED_REGISTER_SIZE));
-      else
-       return read_register (A3_REGNUM);
-    }
-  else
-    {
-      int adjust = saved_regs_size (fi);
+  trad_frame_get_id (cache, this_id);
+}
 
-      /* Our caller does not have a frame pointer.  So his frame starts
-         at the base of our frame (fi->frame) + register save space
-         + <his size>.  */
-      return get_frame_base (fi) + adjust + -get_frame_extra_info (dummy)->stack_size;
-    }
+static void
+mn10300_frame_prev_register (struct frame_info *next_frame,
+                            void **this_prologue_cache,
+                            int regnum, int *optimizedp,
+                            enum lval_type *lvalp, CORE_ADDR *addrp,
+                            int *realnump, void *bufferp)
+{
+  struct trad_frame_cache *cache =
+    mn10300_frame_unwind_cache (next_frame, this_prologue_cache);
+
+  trad_frame_get_register (cache, next_frame, regnum, optimizedp, 
+                          lvalp, addrp, realnump, bufferp);
+  /* Or...
+  trad_frame_get_prev_register (next_frame, cache->prev_regs, regnum, 
+                          optimizedp, lvalp, addrp, realnump, bufferp);
+  */
 }
 
-/* Function: skip_prologue
-   Return the address of the first inst past the prologue of the function.  */
+static const struct frame_unwind mn10300_frame_unwind = {
+  NORMAL_FRAME,
+  mn10300_frame_this_id, 
+  mn10300_frame_prev_register
+};
 
 static CORE_ADDR
-mn10300_skip_prologue (CORE_ADDR pc)
+mn10300_frame_base_address (struct frame_info *next_frame,
+                           void **this_prologue_cache)
 {
-  /* We used to check the debug symbols, but that can lose if
-     we have a null prologue.  */
-  return mn10300_analyze_prologue (NULL, pc);
+  struct trad_frame_cache *cache = 
+    mn10300_frame_unwind_cache (next_frame, this_prologue_cache);
+
+  return trad_frame_get_this_base (cache);
 }
 
-/* generic_pop_current_frame calls this function if the current
-   frame isn't a dummy frame.  */
-static void
-mn10300_pop_frame_regular (struct frame_info *frame)
+static const struct frame_unwind *
+mn10300_frame_sniffer (struct frame_info *next_frame)
 {
-  int regnum;
+  return &mn10300_frame_unwind;
+}
 
-  write_register (PC_REGNUM, DEPRECATED_FRAME_SAVED_PC (frame));
+static const struct frame_base mn10300_frame_base = {
+  &mn10300_frame_unwind, 
+  mn10300_frame_base_address, 
+  mn10300_frame_base_address,
+  mn10300_frame_base_address
+};
 
-  /* Restore any saved registers.  */
-  for (regnum = 0; regnum < NUM_REGS; regnum++)
-    if (get_frame_saved_regs (frame)[regnum] != 0)
-      {
-        ULONGEST value;
+static CORE_ADDR
+mn10300_unwind_pc (struct gdbarch *gdbarch, struct frame_info *next_frame)
+{
+  ULONGEST pc;
 
-        value = read_memory_unsigned_integer (get_frame_saved_regs (frame)[regnum],
-                                              REGISTER_RAW_SIZE (regnum));
-        write_register (regnum, value);
-      }
+  frame_unwind_unsigned_register (next_frame, E_PC_REGNUM, &pc);
+  return pc;
+}
 
-  /* Actually cut back the stack.  */
-  write_register (SP_REGNUM, get_frame_base (frame));
+static CORE_ADDR
+mn10300_unwind_sp (struct gdbarch *gdbarch, struct frame_info *next_frame)
+{
+  ULONGEST sp;
 
-  /* Don't we need to set the PC?!?  XXX FIXME.  */
+  frame_unwind_unsigned_register (next_frame, E_SP_REGNUM, &sp);
+  return sp;
 }
 
-/* Function: pop_frame
-   This routine gets called when either the user uses the `return'
-   command, or the call dummy breakpoint gets hit.  */
 static void
-mn10300_pop_frame (void)
+mn10300_frame_unwind_init (struct gdbarch *gdbarch)
 {
-  /* This function checks for and handles generic dummy frames, and
-     calls back to our function for ordinary frames.  */
-  generic_pop_current_frame (mn10300_pop_frame_regular);
-
-  /* Throw away any cached frame information.  */
-  flush_cached_frames ();
+  frame_unwind_append_sniffer (gdbarch, dwarf2_frame_sniffer);
+  frame_unwind_append_sniffer (gdbarch, mn10300_frame_sniffer);
+  frame_base_set_default (gdbarch, &mn10300_frame_base);
+  set_gdbarch_unwind_dummy_id (gdbarch, mn10300_unwind_dummy_id);
+  set_gdbarch_unwind_pc (gdbarch, mn10300_unwind_pc);
+  set_gdbarch_unwind_sp (gdbarch, mn10300_unwind_sp);
 }
 
-/* Function: push_arguments
-   Setup arguments for a call to the target.  Arguments go in
-   order on the stack.  */
+/* Function: push_dummy_call
+ *
+ * Set up machine state for a target call, including
+ * function arguments, stack, return address, etc.
+ *
+ */
 
 static CORE_ADDR
-mn10300_push_arguments (int nargs, struct value **args, CORE_ADDR sp,
-                       int struct_return, CORE_ADDR struct_addr)
+mn10300_push_dummy_call (struct gdbarch *gdbarch, 
+                        struct value *target_func,
+                        struct regcache *regcache,
+                        CORE_ADDR bp_addr, 
+                        int nargs, struct value **args,
+                        CORE_ADDR sp, 
+                        int struct_return,
+                        CORE_ADDR struct_addr)
 {
-  int argnum = 0;
-  int len = 0;
+  const int push_size = register_size (gdbarch, E_PC_REGNUM);
+  int regs_used;
+  int len, arg_len; 
   int stack_offset = 0;
-  int regsused = struct_return ? 1 : 0;
+  int argnum;
+  char *val, valbuf[MAX_REGISTER_SIZE];
 
   /* This should be a nop, but align the stack just in case something
      went wrong.  Stacks are four byte aligned on the mn10300.  */
@@ -784,56 +868,63 @@ mn10300_push_arguments (int nargs, struct value **args, CORE_ADDR sp,
 
      XXX This doesn't appear to handle pass-by-invisible reference
      arguments.  */
-  for (argnum = 0; argnum < nargs; argnum++)
+  regs_used = struct_return ? 1 : 0;
+  for (len = 0, argnum = 0; argnum < nargs; argnum++)
     {
-      int arg_length = (TYPE_LENGTH (VALUE_TYPE (args[argnum])) + 3) & ~3;
-
-      while (regsused < 2 && arg_length > 0)
+      arg_len = (TYPE_LENGTH (value_type (args[argnum])) + 3) & ~3;
+      while (regs_used < 2 && arg_len > 0)
        {
-         regsused++;
-         arg_length -= 4;
+         regs_used++;
+         arg_len -= push_size;
        }
-      len += arg_length;
+      len += arg_len;
     }
 
   /* Allocate stack space.  */
   sp -= len;
 
-  regsused = struct_return ? 1 : 0;
+  if (struct_return)
+    {
+      regs_used = 1;
+      write_register (E_D0_REGNUM, struct_addr);
+    }
+  else
+    regs_used = 0;
+
   /* Push all arguments onto the stack. */
   for (argnum = 0; argnum < nargs; argnum++)
     {
-      int len;
-      char *val;
-
-      /* XXX Check this.  What about UNIONS?  */
-      if (TYPE_CODE (VALUE_TYPE (*args)) == TYPE_CODE_STRUCT
-         && TYPE_LENGTH (VALUE_TYPE (*args)) > 8)
+      /* FIXME what about structs?  Unions?  */
+      if (TYPE_CODE (value_type (*args)) == TYPE_CODE_STRUCT
+         && TYPE_LENGTH (value_type (*args)) > 8)
        {
-         /* XXX Wrong, we want a pointer to this argument.  */
-         len = TYPE_LENGTH (VALUE_TYPE (*args));
-         val = (char *) VALUE_CONTENTS (*args);
+         /* Change to pointer-to-type.  */
+         arg_len = push_size;
+         store_unsigned_integer (valbuf, push_size, 
+                                 VALUE_ADDRESS (*args));
+         val = &valbuf[0];
        }
       else
        {
-         len = TYPE_LENGTH (VALUE_TYPE (*args));
-         val = (char *) VALUE_CONTENTS (*args);
+         arg_len = TYPE_LENGTH (value_type (*args));
+         val = (char *) value_contents (*args);
        }
 
-      while (regsused < 2 && len > 0)
+      while (regs_used < 2 && arg_len > 0)
        {
-         write_register (regsused, extract_unsigned_integer (val, 4));
-         val += 4;
-         len -= 4;
-         regsused++;
+         write_register (regs_used, 
+                         extract_unsigned_integer (val, push_size));
+         val += push_size;
+         arg_len -= push_size;
+         regs_used++;
        }
 
-      while (len > 0)
+      while (arg_len > 0)
        {
-         write_memory (sp + stack_offset, val, 4);
-         len -= 4;
-         val += 4;
-         stack_offset += 4;
+         write_memory (sp + stack_offset, val, push_size);
+         arg_len -= push_size;
+         val += push_size;
+         stack_offset += push_size;
        }
 
       args++;
@@ -841,384 +932,97 @@ mn10300_push_arguments (int nargs, struct value **args, CORE_ADDR sp,
 
   /* Make space for the flushback area.  */
   sp -= 8;
-  return sp;
-}
 
-/* Function: push_return_address (pc)
-   Set up the return address for the inferior function call.
-   Needed for targets where we don't actually execute a JSR/BSR instruction */
-
-static CORE_ADDR
-mn10300_push_return_address (CORE_ADDR pc, CORE_ADDR sp)
-{
-  unsigned char buf[4];
-
-  store_unsigned_integer (buf, 4, CALL_DUMMY_ADDRESS ());
-  write_memory (sp - 4, buf, 4);
-  return sp - 4;
-}
-
-/* Function: store_struct_return (addr,sp)
-   Store the structure value return address for an inferior function
-   call.  */
-
-static void
-mn10300_store_struct_return (CORE_ADDR addr, CORE_ADDR sp)
-{
-  /* The structure return address is passed as the first argument.  */
-  write_register (0, addr);
-}
-
-/* Function: frame_saved_pc 
-   Find the caller of this frame.  We do this by seeing if RP_REGNUM
-   is saved in the stack anywhere, otherwise we get it from the
-   registers.  If the inner frame is a dummy frame, return its PC
-   instead of RP, because that's where "caller" of the dummy-frame
-   will be found.  */
-
-static CORE_ADDR
-mn10300_frame_saved_pc (struct frame_info *fi)
-{
-  int adjust = saved_regs_size (fi);
-
-  return (read_memory_integer (get_frame_base (fi) + adjust,
-                              DEPRECATED_REGISTER_SIZE));
-}
-
-/* Function: mn10300_init_extra_frame_info
-   Setup the frame's frame pointer, pc, and frame addresses for saved
-   registers.  Most of the work is done in mn10300_analyze_prologue().
-
-   Note that when we are called for the last frame (currently active frame),
-   that get_frame_pc (fi) and fi->frame will already be setup.  However, fi->frame will
-   be valid only if this routine uses FP.  For previous frames, fi-frame will
-   always be correct.  mn10300_analyze_prologue will fix fi->frame if
-   it's not valid.
-
-   We can be called with the PC in the call dummy under two
-   circumstances.  First, during normal backtracing, second, while
-   figuring out the frame pointer just prior to calling the target
-   function (see call_function_by_hand).  */
-
-static void
-mn10300_init_extra_frame_info (int fromleaf, struct frame_info *fi)
-{
-  if (get_next_frame (fi))
-    deprecated_update_frame_pc_hack (fi, DEPRECATED_FRAME_SAVED_PC (get_next_frame (fi)));
-
-  frame_saved_regs_zalloc (fi);
-  frame_extra_info_zalloc (fi, sizeof (struct frame_extra_info));
-
-  get_frame_extra_info (fi)->status = 0;
-  get_frame_extra_info (fi)->stack_size = 0;
-
-  mn10300_analyze_prologue (fi, 0);
-}
-
-
-/* This function's job is handled by init_extra_frame_info.  */
-static void
-mn10300_frame_init_saved_regs (struct frame_info *frame)
-{
-}
-
-
-/* Function: mn10300_virtual_frame_pointer
-   Return the register that the function uses for a frame pointer, 
-   plus any necessary offset to be applied to the register before
-   any frame pointer offsets.  */
-
-static void
-mn10300_virtual_frame_pointer (CORE_ADDR pc,
-                              int *reg,
-                              LONGEST *offset)
-{
-  struct frame_info *dummy = analyze_dummy_frame (pc, 0);
-  /* Set up a dummy frame_info, Analyze the prolog and fill in the
-     extra info.  */
-  /* Results will tell us which type of frame it uses.  */
-  if (get_frame_extra_info (dummy)->status & MY_FRAME_IN_SP)
-    {
-      *reg = SP_REGNUM;
-      *offset = -(get_frame_extra_info (dummy)->stack_size);
-    }
-  else
-    {
-      *reg = A3_REGNUM;
-      *offset = 0;
-    }
-}
-
-static int
-mn10300_reg_struct_has_addr (int gcc_p, struct type *type)
-{
-  return (TYPE_LENGTH (type) > 8);
-}
-
-static struct type *
-mn10300_register_virtual_type (int reg)
-{
-  return builtin_type_int;
-}
-
-static int
-mn10300_register_byte (int reg)
-{
-  return (reg * 4);
-}
-
-static int
-mn10300_register_virtual_size (int reg)
-{
-  return 4;
-}
-
-static int
-mn10300_register_raw_size (int reg)
-{
-  return 4;
-}
-
-/* If DWARF2 is a register number appearing in Dwarf2 debug info, then
-   mn10300_dwarf2_reg_to_regnum (DWARF2) is the corresponding GDB
-   register number.  Why don't Dwarf2 and GDB use the same numbering?
-   Who knows?  But since people have object files lying around with
-   the existing Dwarf2 numbering, and other people have written stubs
-   to work with the existing GDB, neither of them can change.  So we
-   just have to cope.  */
-static int
-mn10300_dwarf2_reg_to_regnum (int dwarf2)
-{
-  /* This table is supposed to be shaped like the REGISTER_NAMES
-     initializer in gcc/config/mn10300/mn10300.h.  Registers which
-     appear in GCC's numbering, but have no counterpart in GDB's
-     world, are marked with a -1.  */
-  static int dwarf2_to_gdb[] = {
-    0,  1,  2,  3,  4,  5,  6,  7, -1, 8,
-    15, 16, 17, 18, 19, 20, 21, 22
-  };
-  int gdb;
-
-  if (dwarf2 < 0
-      || dwarf2 >= (sizeof (dwarf2_to_gdb) / sizeof (dwarf2_to_gdb[0]))
-      || dwarf2_to_gdb[dwarf2] == -1)
-    internal_error (__FILE__, __LINE__,
-                    "bogus register number in debug info: %d", dwarf2);
-
-  return dwarf2_to_gdb[dwarf2];
-}
-
-static void
-mn10300_print_register (const char *name, int regnum, int reg_width)
-{
-  char raw_buffer[MAX_REGISTER_SIZE];
-
-  if (reg_width)
-    printf_filtered ("%*s: ", reg_width, name);
-  else
-    printf_filtered ("%s: ", name);
-
-  /* Get the data */
-  if (!frame_register_read (deprecated_selected_frame, regnum, raw_buffer))
-    {
-      printf_filtered ("[invalid]");
-      return;
-    }
-  else
-    {
-      int byte;
-      if (TARGET_BYTE_ORDER == BFD_ENDIAN_BIG)
-       {
-         for (byte = REGISTER_RAW_SIZE (regnum) - REGISTER_VIRTUAL_SIZE (regnum);
-              byte < REGISTER_RAW_SIZE (regnum);
-              byte++)
-           printf_filtered ("%02x", (unsigned char) raw_buffer[byte]);
-       }
-      else
-       {
-         for (byte = REGISTER_VIRTUAL_SIZE (regnum) - 1;
-              byte >= 0;
-              byte--)
-           printf_filtered ("%02x", (unsigned char) raw_buffer[byte]);
-       }
-    }
-}
-
-static void
-mn10300_do_registers_info (int regnum, int fpregs)
-{
-  if (regnum >= 0)
-    {
-      const char *name = REGISTER_NAME (regnum);
-      if (name == NULL || name[0] == '\0')
-       error ("Not a valid register for the current processor type");
-      mn10300_print_register (name, regnum, 0);
-      printf_filtered ("\n");
-    }
-  else
-    {
-      /* print registers in an array 4x8 */
-      int r;
-      int reg;
-      const int nr_in_row = 4;
-      const int reg_width = 4;
-      for (r = 0; r < NUM_REGS; r += nr_in_row)
-       {
-         int c;
-         int printing = 0;
-         int padding = 0;
-         for (c = r; c < r + nr_in_row; c++)
-           {
-             const char *name = REGISTER_NAME (c);
-             if (name != NULL && *name != '\0')
-               {
-                 printing = 1;
-                 while (padding > 0)
-                   {
-                     printf_filtered (" ");
-                     padding--;
-                   }
-                 mn10300_print_register (name, c, reg_width);
-                 printf_filtered (" ");
-               }
-             else
-               {
-                 padding += (reg_width + 2 + 8 + 1);
-               }
-           }
-         if (printing)
-           printf_filtered ("\n");
-       }
-    }
-}
-
-static CORE_ADDR
-mn10300_read_fp (void)
-{
-  /* That's right, we're using the stack pointer as our frame pointer.  */
-  gdb_assert (SP_REGNUM >= 0);
-  return read_register (SP_REGNUM);
+  /* Push the return address that contains the magic breakpoint.  */
+  sp -= 4;
+  write_memory_unsigned_integer (sp, push_size, bp_addr);
+  /* Update $sp.  */
+  regcache_cooked_write_unsigned (regcache, E_SP_REGNUM, sp);
+  return sp;
 }
 
-/* Dump out the mn10300 speciic architecture information. */
-
-static void
-mn10300_dump_tdep (struct gdbarch *current_gdbarch, struct ui_file *file)
-{
-  struct gdbarch_tdep *tdep = gdbarch_tdep (current_gdbarch);
-  fprintf_unfiltered (file, "mn10300_dump_tdep: am33_mode = %d\n",
-                     tdep->am33_mode);
-}
 
 static struct gdbarch *
 mn10300_gdbarch_init (struct gdbarch_info info,
                      struct gdbarch_list *arches)
 {
-  static LONGEST mn10300_call_dummy_words[] = { 0 };
   struct gdbarch *gdbarch;
-  struct gdbarch_tdep *tdep = NULL;
-  int am33_mode;
-  gdbarch_register_name_ftype *register_name;
-  int mach;
-  int num_regs;
+  struct gdbarch_tdep *tdep;
 
   arches = gdbarch_list_lookup_by_info (arches, &info);
   if (arches != NULL)
     return arches->gdbarch;
+
   tdep = xmalloc (sizeof (struct gdbarch_tdep));
   gdbarch = gdbarch_alloc (&info, tdep);
 
-  if (info.bfd_arch_info != NULL
-      && info.bfd_arch_info->arch == bfd_arch_mn10300)
-    mach = info.bfd_arch_info->mach;
-  else
-    mach = 0;
-  switch (mach)
+  switch (info.bfd_arch_info->mach)
     {
     case 0:
     case bfd_mach_mn10300:
-      am33_mode = 0;
-      register_name = mn10300_generic_register_name;
-      num_regs = 32;
+      set_gdbarch_register_name (gdbarch, mn10300_generic_register_name);
+      tdep->am33_mode = 0;
       break;
     case bfd_mach_am33:
-      am33_mode = 1;
-      register_name = am33_register_name;
-      num_regs = 32;
+      set_gdbarch_register_name (gdbarch, am33_register_name);
+      tdep->am33_mode = 1;
       break;
     default:
       internal_error (__FILE__, __LINE__,
-                     "mn10300_gdbarch_init: Unknown mn10300 variant");
-      return NULL; /* keep GCC happy. */
+                     _("mn10300_gdbarch_init: Unknown mn10300 variant"));
+      break;
     }
 
   /* Registers.  */
-  set_gdbarch_num_regs (gdbarch, num_regs);
-  set_gdbarch_register_name (gdbarch, register_name);
-  set_gdbarch_deprecated_register_size (gdbarch, 4);
-  set_gdbarch_deprecated_register_bytes (gdbarch, num_regs * gdbarch_deprecated_register_size (gdbarch));
-  set_gdbarch_deprecated_max_register_raw_size (gdbarch, 4);
-  set_gdbarch_register_raw_size (gdbarch, mn10300_register_raw_size);
-  set_gdbarch_register_byte (gdbarch, mn10300_register_byte);
-  set_gdbarch_deprecated_max_register_virtual_size (gdbarch, 4);
-  set_gdbarch_register_virtual_size (gdbarch, mn10300_register_virtual_size);
-  set_gdbarch_register_virtual_type (gdbarch, mn10300_register_virtual_type);
-  set_gdbarch_dwarf2_reg_to_regnum (gdbarch, mn10300_dwarf2_reg_to_regnum);
-  set_gdbarch_deprecated_do_registers_info (gdbarch, mn10300_do_registers_info);
-  set_gdbarch_sp_regnum (gdbarch, 8);
-  set_gdbarch_pc_regnum (gdbarch, 9);
-  set_gdbarch_deprecated_fp_regnum (gdbarch, 31);
-  set_gdbarch_virtual_frame_pointer (gdbarch, mn10300_virtual_frame_pointer);
+  set_gdbarch_num_regs (gdbarch, E_NUM_REGS);
+  set_gdbarch_register_type (gdbarch, mn10300_register_type);
+  set_gdbarch_skip_prologue (gdbarch, mn10300_skip_prologue);
+  set_gdbarch_read_pc (gdbarch, mn10300_read_pc);
+  set_gdbarch_write_pc (gdbarch, mn10300_write_pc);
+  set_gdbarch_pc_regnum (gdbarch, E_PC_REGNUM);
+  set_gdbarch_sp_regnum (gdbarch, E_SP_REGNUM);
 
+  /* Stack unwinding.  */
+  set_gdbarch_inner_than (gdbarch, core_addr_lessthan);
   /* Breakpoints.  */
   set_gdbarch_breakpoint_from_pc (gdbarch, mn10300_breakpoint_from_pc);
-  set_gdbarch_function_start_offset (gdbarch, 0);
-  set_gdbarch_decr_pc_after_break (gdbarch, 0);
+  /* decr_pc_after_break? */
+  /* Disassembly.  */
+  set_gdbarch_print_insn (gdbarch, print_insn_mn10300);
+
+  /* Stage 2 */
+  /* MVS Note: at least the first one is deprecated!  */
+  set_gdbarch_deprecated_use_struct_convention (gdbarch, 
+                                               mn10300_use_struct_convention);
+  set_gdbarch_store_return_value (gdbarch, mn10300_store_return_value);
+  set_gdbarch_extract_return_value (gdbarch, mn10300_extract_return_value);
+  
+  /* Stage 3 -- get target calls working.  */
+  set_gdbarch_push_dummy_call (gdbarch, mn10300_push_dummy_call);
+  /* set_gdbarch_return_value (store, extract) */
 
-  /* Stack unwinding.  */
-  set_gdbarch_inner_than (gdbarch, core_addr_lessthan);
-  set_gdbarch_deprecated_saved_pc_after_call (gdbarch, mn10300_saved_pc_after_call);
-  set_gdbarch_deprecated_init_extra_frame_info (gdbarch, mn10300_init_extra_frame_info);
-  set_gdbarch_deprecated_init_frame_pc (gdbarch, init_frame_pc_noop);
-  set_gdbarch_deprecated_frame_init_saved_regs (gdbarch, mn10300_frame_init_saved_regs);
-  set_gdbarch_deprecated_frame_chain (gdbarch, mn10300_frame_chain);
-  set_gdbarch_deprecated_frame_saved_pc (gdbarch, mn10300_frame_saved_pc);
-  set_gdbarch_deprecated_extract_return_value (gdbarch, mn10300_extract_return_value);
-  set_gdbarch_deprecated_extract_struct_value_address
-    (gdbarch, mn10300_extract_struct_value_address);
-  set_gdbarch_deprecated_store_return_value (gdbarch, mn10300_store_return_value);
-  set_gdbarch_deprecated_store_struct_return (gdbarch, mn10300_store_struct_return);
-  set_gdbarch_deprecated_pop_frame (gdbarch, mn10300_pop_frame);
-  set_gdbarch_skip_prologue (gdbarch, mn10300_skip_prologue);
-  set_gdbarch_frame_args_skip (gdbarch, 0);
-  set_gdbarch_frame_num_args (gdbarch, frame_num_args_unknown);
-  /* That's right, we're using the stack pointer as our frame pointer.  */
-  set_gdbarch_deprecated_target_read_fp (gdbarch, mn10300_read_fp);
-
-  /* Calling functions in the inferior from GDB.  */
-  set_gdbarch_deprecated_call_dummy_words (gdbarch, mn10300_call_dummy_words);
-  set_gdbarch_deprecated_sizeof_call_dummy_words (gdbarch, sizeof (mn10300_call_dummy_words));
-  set_gdbarch_deprecated_pc_in_call_dummy (gdbarch, deprecated_pc_in_call_dummy_at_entry_point);
-  set_gdbarch_deprecated_push_arguments (gdbarch, mn10300_push_arguments);
-  set_gdbarch_reg_struct_has_addr (gdbarch, mn10300_reg_struct_has_addr);
-  set_gdbarch_deprecated_push_return_address (gdbarch, mn10300_push_return_address);
-  set_gdbarch_save_dummy_frame_tos (gdbarch, generic_save_dummy_frame_tos);
-  set_gdbarch_use_struct_convention (gdbarch, mn10300_use_struct_convention);
-
-  tdep->am33_mode = am33_mode;
-
-  /* Should be using push_dummy_call.  */
-  set_gdbarch_deprecated_dummy_write_sp (gdbarch, deprecated_write_sp);
+
+  mn10300_frame_unwind_init (gdbarch);
+
+  /* Hook in ABI-specific overrides, if they have been registered.  */
+  gdbarch_init_osabi (info, gdbarch);
 
   return gdbarch;
 }
  
+/* Dump out the mn10300 specific architecture information. */
+
+static void
+mn10300_dump_tdep (struct gdbarch *current_gdbarch, struct ui_file *file)
+{
+  struct gdbarch_tdep *tdep = gdbarch_tdep (current_gdbarch);
+  fprintf_unfiltered (file, "mn10300_dump_tdep: am33_mode = %d\n",
+                     tdep->am33_mode);
+}
+
 void
 _initialize_mn10300_tdep (void)
 {
-/*  printf("_initialize_mn10300_tdep\n"); */
-
-  deprecated_tm_print_insn = print_insn_mn10300;
-
-  register_gdbarch_init (bfd_arch_mn10300, mn10300_gdbarch_init);
+  gdbarch_register (bfd_arch_mn10300, mn10300_gdbarch_init, mn10300_dump_tdep);
 }
+
This page took 0.040394 seconds and 4 git commands to generate.