* mips-tdep.c (mips16_scan_prologue): Replace read_next_frame_reg
[deliverable/binutils-gdb.git] / gdb / v850-tdep.c
index 39701458ed4215774e7e78211568e4dc15821aac..7aa7dc33bbcbfd7cc53acfe7f08453ddce77bc12 100644 (file)
@@ -1,5 +1,6 @@
 /* Target-dependent code for the NEC V850 for GDB, the GNU debugger.
-   Copyright 1996, 1998, 1999, 2000, 2001, 2002, 2003
+
+   Copyright (C) 1996, 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2007
    Free Software Foundation, Inc.
 
    This file is part of GDB.
 
    You should have received a copy of the GNU General Public License
    along with this program; if not, write to the Free Software
-   Foundation, Inc., 59 Temple Place - Suite 330,
-   Boston, MA 02111-1307, USA.  */
+   Foundation, Inc., 51 Franklin Street, Fifth Floor,
+   Boston, MA 02110-1301, USA.  */
 
 #include "defs.h"
 #include "frame.h"
+#include "frame-base.h"
+#include "trad-frame.h"
+#include "frame-unwind.h"
+#include "dwarf2-frame.h"
+#include "gdbtypes.h"
 #include "inferior.h"
-#include "target.h"
-#include "value.h"
-#include "bfd.h"
 #include "gdb_string.h"
+#include "gdb_assert.h"
 #include "gdbcore.h"
-#include "symfile.h"
 #include "arch-utils.h"
 #include "regcache.h"
-#include "symtab.h"
-
-struct gdbarch_tdep
-{
-  /* gdbarch target dependent data here. Currently unused for v850. */
-};
-
-/* Extra info which is saved in each frame_info. */
-struct frame_extra_info
-{ 
-};
-
-enum {
- E_R0_REGNUM,
- E_R1_REGNUM,
- E_R2_REGNUM, E_SAVE1_START_REGNUM = E_R2_REGNUM, E_SAVE1_END_REGNUM = E_R2_REGNUM,
- E_R3_REGNUM, E_SP_REGNUM = E_R3_REGNUM,
- E_R4_REGNUM,
- E_R5_REGNUM,
- E_R6_REGNUM, E_ARG0_REGNUM = E_R6_REGNUM,
- E_R7_REGNUM,
- E_R8_REGNUM,
- E_R9_REGNUM, E_ARGLAST_REGNUM = E_R9_REGNUM,
- E_R10_REGNUM, E_V0_REGNUM = E_R10_REGNUM,
- E_R11_REGNUM, E_V1_REGNUM = E_R11_REGNUM,
- E_R12_REGNUM,
- E_R13_REGNUM,
- E_R14_REGNUM,
- E_R15_REGNUM,
- E_R16_REGNUM,
- E_R17_REGNUM,
- E_R18_REGNUM,
- E_R19_REGNUM,
- E_R20_REGNUM, E_SAVE2_START_REGNUM = E_R20_REGNUM,
- E_R21_REGNUM,
- E_R22_REGNUM,
- E_R23_REGNUM,
- E_R24_REGNUM,
- E_R25_REGNUM,
- E_R26_REGNUM,
- E_R27_REGNUM,
- E_R28_REGNUM,
- E_R29_REGNUM, E_SAVE2_END_REGNUM = E_R29_REGNUM, E_FP_RAW_REGNUM = E_R29_REGNUM,
- E_R30_REGNUM, E_EP_REGNUM = E_R30_REGNUM,
- E_R31_REGNUM, E_SAVE3_START_REGNUM = E_R31_REGNUM, E_SAVE3_END_REGNUM = E_R31_REGNUM, E_RP_REGNUM = E_R31_REGNUM,
- E_R32_REGNUM, E_SR0_REGNUM = E_R32_REGNUM,
- E_R33_REGNUM,
- E_R34_REGNUM,
- E_R35_REGNUM,
- E_R36_REGNUM,
- E_R37_REGNUM, E_PS_REGNUM = E_R37_REGNUM,
- E_R38_REGNUM,
- E_R39_REGNUM,
- E_R40_REGNUM,
- E_R41_REGNUM,
- E_R42_REGNUM,
- E_R43_REGNUM,
- E_R44_REGNUM,
- E_R45_REGNUM,
- E_R46_REGNUM,
- E_R47_REGNUM,
- E_R48_REGNUM,
- E_R49_REGNUM,
- E_R50_REGNUM,
- E_R51_REGNUM,
- E_R52_REGNUM, E_CTBP_REGNUM = E_R52_REGNUM,
- E_R53_REGNUM,
- E_R54_REGNUM,
- E_R55_REGNUM,
- E_R56_REGNUM,
- E_R57_REGNUM,
- E_R58_REGNUM,
- E_R59_REGNUM,
- E_R60_REGNUM,
- E_R61_REGNUM,
- E_R62_REGNUM,
- E_R63_REGNUM,
- E_R64_REGNUM, E_PC_REGNUM = E_R64_REGNUM,
- E_R65_REGNUM, E_FP_REGNUM = E_R65_REGNUM,
- E_NUM_REGS
-};
+#include "dis-asm.h"
+#include "osabi.h"
 
 enum
-{
-  v850_reg_size = 4
-};
+  {
+    E_R0_REGNUM,
+    E_R1_REGNUM,
+    E_R2_REGNUM,
+    E_R3_REGNUM, E_SP_REGNUM = E_R3_REGNUM,
+    E_R4_REGNUM,
+    E_R5_REGNUM,
+    E_R6_REGNUM, E_ARG0_REGNUM = E_R6_REGNUM,
+    E_R7_REGNUM,
+    E_R8_REGNUM,
+    E_R9_REGNUM, E_ARGLAST_REGNUM = E_R9_REGNUM,
+    E_R10_REGNUM, E_V0_REGNUM = E_R10_REGNUM,
+    E_R11_REGNUM, E_V1_REGNUM = E_R11_REGNUM,
+    E_R12_REGNUM,
+    E_R13_REGNUM,
+    E_R14_REGNUM,
+    E_R15_REGNUM,
+    E_R16_REGNUM,
+    E_R17_REGNUM,
+    E_R18_REGNUM,
+    E_R19_REGNUM,
+    E_R20_REGNUM,
+    E_R21_REGNUM,
+    E_R22_REGNUM,
+    E_R23_REGNUM,
+    E_R24_REGNUM,
+    E_R25_REGNUM,
+    E_R26_REGNUM,
+    E_R27_REGNUM,
+    E_R28_REGNUM,
+    E_R29_REGNUM, E_FP_REGNUM = E_R29_REGNUM,
+    E_R30_REGNUM, E_EP_REGNUM = E_R30_REGNUM,
+    E_R31_REGNUM, E_LP_REGNUM = E_R31_REGNUM,
+    E_R32_REGNUM, E_SR0_REGNUM = E_R32_REGNUM,
+    E_R33_REGNUM,
+    E_R34_REGNUM,
+    E_R35_REGNUM,
+    E_R36_REGNUM,
+    E_R37_REGNUM, E_PS_REGNUM = E_R37_REGNUM,
+    E_R38_REGNUM,
+    E_R39_REGNUM,
+    E_R40_REGNUM,
+    E_R41_REGNUM,
+    E_R42_REGNUM,
+    E_R43_REGNUM,
+    E_R44_REGNUM,
+    E_R45_REGNUM,
+    E_R46_REGNUM,
+    E_R47_REGNUM,
+    E_R48_REGNUM,
+    E_R49_REGNUM,
+    E_R50_REGNUM,
+    E_R51_REGNUM,
+    E_R52_REGNUM, E_CTBP_REGNUM = E_R52_REGNUM,
+    E_R53_REGNUM,
+    E_R54_REGNUM,
+    E_R55_REGNUM,
+    E_R56_REGNUM,
+    E_R57_REGNUM,
+    E_R58_REGNUM,
+    E_R59_REGNUM,
+    E_R60_REGNUM,
+    E_R61_REGNUM,
+    E_R62_REGNUM,
+    E_R63_REGNUM,
+    E_R64_REGNUM, E_PC_REGNUM = E_R64_REGNUM,
+    E_R65_REGNUM,
+    E_NUM_REGS
+  };
 
-/* Size of all registers as a whole. */
 enum
 {
-  E_ALL_REGS_SIZE = (E_NUM_REGS) * v850_reg_size
+  v850_reg_size = 4
 };
 
-/* Size of return datatype which fits into all return registers. */
+/* Size of return datatype which fits into all return registers.  */
 enum
 {
   E_MAX_RETTYPE_SIZE_IN_REGS = 2 * v850_reg_size
 };
 
-static LONGEST call_dummy_nil[] = {0};
-
-static char *v850_generic_reg_names[] =
-{ "r0", "r1", "r2", "r3", "r4", "r5", "r6", "r7", 
-  "r8", "r9", "r10", "r11", "r12", "r13", "r14", "r15", 
-  "r16", "r17", "r18", "r19", "r20", "r21", "r22", "r23", 
-  "r24", "r25", "r26", "r27", "r28", "r29", "r30", "r31",
-  "eipc", "eipsw", "fepc", "fepsw", "ecr", "psw", "sr6", "sr7",
-  "sr8", "sr9", "sr10", "sr11", "sr12", "sr13", "sr14", "sr15",
-  "sr16", "sr17", "sr18", "sr19", "sr20", "sr21", "sr22", "sr23",
-  "sr24", "sr25", "sr26", "sr27", "sr28", "sr29", "sr30", "sr31",
-  "pc", "fp"
-};
-
-static char *v850e_reg_names[] =
-{
-  "r0", "r1", "r2", "r3", "r4", "r5", "r6", "r7",
-  "r8", "r9", "r10", "r11", "r12", "r13", "r14", "r15",
-  "r16", "r17", "r18", "r19", "r20", "r21", "r22", "r23",
-  "r24", "r25", "r26", "r27", "r28", "r29", "r30", "r31",
-  "eipc", "eipsw", "fepc", "fepsw", "ecr", "psw", "sr6", "sr7",
-  "sr8", "sr9", "sr10", "sr11", "sr12", "sr13", "sr14", "sr15",
-  "ctpc", "ctpsw", "dbpc", "dbpsw", "ctbp", "sr21", "sr22", "sr23",
-  "sr24", "sr25", "sr26", "sr27", "sr28", "sr29", "sr30", "sr31",
-  "pc", "fp"
+struct v850_frame_cache
+{ 
+  /* Base address.  */
+  CORE_ADDR base;
+  LONGEST sp_offset;
+  CORE_ADDR pc;
+  
+  /* Flag showing that a frame has been created in the prologue code.  */
+  int uses_fp;
+  
+  /* Saved registers.  */
+  struct trad_frame_saved_reg *saved_regs;
 };
 
-char **v850_register_names = v850_generic_reg_names;
-
-struct
-  {
-    char **regnames;
-    int mach;
-  }
-v850_processor_type_table[] =
+/* Info gleaned from scanning a function's prologue.  */
+struct pifsr           /* Info about one saved register.  */
 {
-  {
-    v850_generic_reg_names, bfd_mach_v850
-  }
-  ,
-  {
-    v850e_reg_names, bfd_mach_v850e
-  }
-  ,
-  {
-    NULL, 0
-  }
+  int offset;          /* Offset from sp or fp.  */
+  int cur_frameoffset; /* Current frameoffset.  */
+  int reg;             /* Saved register number.  */
 };
 
-/* Info gleaned from scanning a function's prologue.  */
-
-struct pifsr                   /* Info about one saved reg */
-  {
-    int framereg;              /* Frame reg (SP or FP) */
-    int offset;                        /* Offset from framereg */
-    int cur_frameoffset;       /* Current frameoffset */
-    int reg;                   /* Saved register number */
-  };
-
-struct prologue_info
-  {
-    int framereg;
-    int frameoffset;
-    int start_function;
-    struct pifsr *pifsrs;
-  };
-
-static CORE_ADDR v850_scan_prologue (CORE_ADDR pc, struct prologue_info *fs);
-
-/* Function: v850_register_name
-   Returns the name of the v850/v850e register N. */
-
 static const char *
 v850_register_name (int regnum)
 {
+  static const char *v850_reg_names[] =
+  { "r0", "r1", "r2", "r3", "r4", "r5", "r6", "r7", 
+    "r8", "r9", "r10", "r11", "r12", "r13", "r14", "r15", 
+    "r16", "r17", "r18", "r19", "r20", "r21", "r22", "r23", 
+    "r24", "r25", "r26", "r27", "r28", "r29", "r30", "r31",
+    "eipc", "eipsw", "fepc", "fepsw", "ecr", "psw", "sr6", "sr7",
+    "sr8", "sr9", "sr10", "sr11", "sr12", "sr13", "sr14", "sr15",
+    "sr16", "sr17", "sr18", "sr19", "sr20", "sr21", "sr22", "sr23",
+    "sr24", "sr25", "sr26", "sr27", "sr28", "sr29", "sr30", "sr31",
+    "pc", "fp"
+  };
   if (regnum < 0 || regnum >= E_NUM_REGS)
-    internal_error (__FILE__, __LINE__,
-                    "v850_register_name: illegal register number %d",
-                    regnum);
-  else
-    return v850_register_names[regnum];
-
-}
-
-/* Function: v850_register_byte 
-   Returns the byte position in the register cache for register N. */
-
-static int
-v850_register_byte (int regnum)
-{
-  if (regnum < 0 || regnum >= E_NUM_REGS)
-    internal_error (__FILE__, __LINE__,
-                    "v850_register_byte: illegal register number %d",
-                    regnum);
-  else
-    return regnum * v850_reg_size;
+    return NULL;
+  return v850_reg_names[regnum];
 }
 
-/* Function: v850_register_raw_size
-   Returns the number of bytes occupied by the register on the target. */
-
-static int
-v850_register_raw_size (int regnum)
+static const char *
+v850e_register_name (int regnum)
 {
+  static const char *v850e_reg_names[] =
+  {
+    "r0", "r1", "r2", "r3", "r4", "r5", "r6", "r7",
+    "r8", "r9", "r10", "r11", "r12", "r13", "r14", "r15",
+    "r16", "r17", "r18", "r19", "r20", "r21", "r22", "r23",
+    "r24", "r25", "r26", "r27", "r28", "r29", "r30", "r31",
+    "eipc", "eipsw", "fepc", "fepsw", "ecr", "psw", "sr6", "sr7",
+    "sr8", "sr9", "sr10", "sr11", "sr12", "sr13", "sr14", "sr15",
+    "ctpc", "ctpsw", "dbpc", "dbpsw", "ctbp", "sr21", "sr22", "sr23",
+    "sr24", "sr25", "sr26", "sr27", "sr28", "sr29", "sr30", "sr31",
+    "pc", "fp"
+  };
   if (regnum < 0 || regnum >= E_NUM_REGS)
-    internal_error (__FILE__, __LINE__,
-                    "v850_register_raw_size: illegal register number %d",
-                    regnum);
-  /* Only the PC has 4 Byte, all other registers 2 Byte. */
-  else
-    return v850_reg_size;
+    return NULL;
+  return v850e_reg_names[regnum];
 }
 
-/* Function: v850_register_virtual_size
-   Returns the number of bytes occupied by the register as represented
-   internally by gdb. */
-
-static int
-v850_register_virtual_size (int regnum)
-{
-  return v850_register_raw_size (regnum);
-}
-
-/* Function: v850_reg_virtual_type 
-   Returns the default type for register N. */
+/* Returns the default type for register N.  */
 
 static struct type *
-v850_reg_virtual_type (int regnum)
+v850_register_type (struct gdbarch *gdbarch, int regnum)
 {
-  if (regnum < 0 || regnum >= E_NUM_REGS)
-    internal_error (__FILE__, __LINE__,
-                    "v850_register_virtual_type: illegal register number %d",
-                    regnum);
-  else if (regnum == E_PC_REGNUM)
-    return builtin_type_uint32;
-  else
-    return builtin_type_int32;
+  if (regnum == E_PC_REGNUM)
+    return builtin_type_void_func_ptr;
+  return builtin_type_int32;
 }
 
 static int
@@ -278,29 +199,22 @@ v850_type_is_scalar (struct type *t)
 
 /* Should call_function allocate stack space for a struct return?  */
 static int
-v850_use_struct_convention (int gcc_p, struct type *type)
+v850_use_struct_convention (struct type *type)
 {
-  /* According to ABI:
-   * return TYPE_LENGTH (type) > 8);
-   */
-
-  /* Current implementation in gcc: */
-
   int i;
   struct type *fld_type, *tgt_type;
 
-  /* 1. The value is greater than 8 bytes -> returned by copying */
+  /* 1. The value is greater than 8 bytes -> returned by copying */
   if (TYPE_LENGTH (type) > 8)
     return 1;
 
-  /* 2. The value is a single basic type -> returned in register */
+  /* 2. The value is a single basic type -> returned in register */
   if (v850_type_is_scalar (type))
     return 0;
 
-  /* The value is a structure or union with a single element
-   * and that element is either a single basic type or an array of
-   * a single basic type whoes size is greater than or equal to 4
-   * -> returned in register */
+  /* The value is a structure or union with a single element and that
+     element is either a single basic type or an array of a single basic
+     type whose size is greater than or equal to 4 -> returned in register.  */
   if ((TYPE_CODE (type) == TYPE_CODE_STRUCT
        || TYPE_CODE (type) == TYPE_CODE_UNION)
        && TYPE_NFIELDS (type) == 1)
@@ -317,9 +231,9 @@ v850_use_struct_convention (int gcc_p, struct type *type)
        }
     }
 
-  /* The value is a structure whose first element is an integer or
-   * a float, and which contains no arrays of more than two elements
-   * -> returned in register */
+  /* The value is a structure whose first element is an integer or a float,
+     and which contains no arrays of more than two elements -> returned in
+     register.  */
   if (TYPE_CODE (type) == TYPE_CODE_STRUCT
       && v850_type_is_scalar (TYPE_FIELD_TYPE (type, 0))
       && TYPE_LENGTH (TYPE_FIELD_TYPE (type, 0)) == 4)
@@ -338,36 +252,33 @@ v850_use_struct_convention (int gcc_p, struct type *type)
       return 0;
     }
     
-  /* The value is a union which contains at least one field which
-   * would be returned in registers according to these rules
-   * -> returned in register */
+  /* The value is a union which contains at least one field which would be
+     returned in registers according to these rules -> returned in register.  */
   if (TYPE_CODE (type) == TYPE_CODE_UNION)
     {
       for (i = 0; i < TYPE_NFIELDS (type); ++i)
         {
          fld_type = TYPE_FIELD_TYPE (type, 0);
-         if (!v850_use_struct_convention (0, fld_type))
+         if (!v850_use_struct_convention (fld_type))
            return 0;
        }
     }
 
   return 1;
 }
-\f
-
 
-/* Structure for mapping bits in register lists to register numbers. */
+/* Structure for mapping bits in register lists to register numbers.  */
 struct reg_list
 {
   long mask;
   int regno;
 };
 
-/* Helper function for v850_scan_prologue to handle prepare instruction. */
+/* Helper function for v850_scan_prologue to handle prepare instruction.  */
 
 static void
-handle_prepare (int insn, int insn2, CORE_ADDR * current_pc_ptr,
-               struct prologue_info *pi, struct pifsr **pifsr_ptr)
+v850_handle_prepare (int insn, int insn2, CORE_ADDR * current_pc_ptr,
+                    struct v850_frame_cache *pi, struct pifsr **pifsr_ptr)
 {
   CORE_ADDR current_pc = *current_pc_ptr;
   struct pifsr *pifsr = *pifsr_ptr;
@@ -392,23 +303,23 @@ handle_prepare (int insn, int insn2, CORE_ADDR * current_pc_ptr,
   };
   int i;
 
-  if ((next & 0x1f) == 0x0b)   /* skip imm16 argument */
+  if ((next & 0x1f) == 0x0b)           /* skip imm16 argument */
     current_pc += 2;
   else if ((next & 0x1f) == 0x13)      /* skip imm16 argument */
     current_pc += 2;
   else if ((next & 0x1f) == 0x1b)      /* skip imm32 argument */
     current_pc += 4;
 
-  /* Calculate the total size of the saved registers, and add it
-     it to the immediate value used to adjust SP. */
+  /* Calculate the total size of the saved registers, and add it to the
+     immediate value used to adjust SP.  */
   for (i = 0; reg_table[i].mask != 0; i++)
     if (list12 & reg_table[i].mask)
-      offset += v850_register_raw_size (reg_table[i].regno);
-  pi->frameoffset -= offset;
+      offset += v850_reg_size;
+  pi->sp_offset -= offset;
 
-  /* Calculate the offsets of the registers relative to the value
-     the SP will have after the registers have been pushed and the
-     imm5 value has been subtracted from it. */
+  /* Calculate the offsets of the registers relative to the value the SP
+     will have after the registers have been pushed and the imm5 value has
+     been subtracted from it.  */
   if (pifsr)
     {
       for (i = 0; reg_table[i].mask != 0; i++)
@@ -416,34 +327,28 @@ handle_prepare (int insn, int insn2, CORE_ADDR * current_pc_ptr,
          if (list12 & reg_table[i].mask)
            {
              int reg = reg_table[i].regno;
-             offset -= v850_register_raw_size (reg);
+             offset -= v850_reg_size;
              pifsr->reg = reg;
              pifsr->offset = offset;
-             pifsr->cur_frameoffset = pi->frameoffset;
-#ifdef DEBUG
-             printf_filtered ("\tSaved register r%d, offset %d", reg, pifsr->offset);
-#endif
+             pifsr->cur_frameoffset = pi->sp_offset;
              pifsr++;
            }
        }
     }
-#ifdef DEBUG
-  printf_filtered ("\tfound ctret after regsave func");
-#endif
 
-  /* Set result parameters. */
+  /* Set result parameters.  */
   *current_pc_ptr = current_pc;
   *pifsr_ptr = pifsr;
 }
 
 
 /* Helper function for v850_scan_prologue to handle pushm/pushl instructions.
-   FIXME: the SR bit of the register list is not supported; must check
-   that the compiler does not ever generate this bit. */
+   The SR bit of the register list is not supported.  gcc does not generate
+   this bit.  */
 
 static void
-handle_pushm (int insn, int insn2, struct prologue_info *pi,
-             struct pifsr **pifsr_ptr)
+v850_handle_pushm (int insn, int insn2, struct v850_frame_cache *pi,
+                  struct pifsr **pifsr_ptr)
 {
   struct pifsr *pifsr = *pifsr_ptr;
   long list12 = ((insn & 0x0f) << 16) + (insn2 & 0xfff0);
@@ -491,22 +396,22 @@ handle_pushm (int insn, int insn2, struct prologue_info *pi,
   struct reg_list *reg_table;
   int i;
 
-  /* Is this a pushml or a pushmh? */
+  /* Is this a pushml or a pushmh?  */
   if ((insn2 & 7) == 1)
     reg_table = pushml_reg_table;
   else
     reg_table = pushmh_reg_table;
 
-  /* Calculate the total size of the saved registers, and add it
-     it to the immediate value used to adjust SP. */
+  /* Calculate the total size of the saved registers, and add it it to the
+     immediate value used to adjust SP.  */
   for (i = 0; reg_table[i].mask != 0; i++)
     if (list12 & reg_table[i].mask)
-      offset += v850_register_raw_size (reg_table[i].regno);
-  pi->frameoffset -= offset;
+      offset += v850_reg_size;
+  pi->sp_offset -= offset;
 
-  /* Calculate the offsets of the registers relative to the value
-     the SP will have after the registers have been pushed and the
-     imm5 value is subtracted from it. */
+  /* Calculate the offsets of the registers relative to the value the SP
+     will have after the registers have been pushed and the imm5 value is
+     subtracted from it.  */
   if (pifsr)
     {
       for (i = 0; reg_table[i].mask != 0; i++)
@@ -514,30 +419,34 @@ handle_pushm (int insn, int insn2, struct prologue_info *pi,
          if (list12 & reg_table[i].mask)
            {
              int reg = reg_table[i].regno;
-             offset -= v850_register_raw_size (reg);
+             offset -= v850_reg_size;
              pifsr->reg = reg;
              pifsr->offset = offset;
-             pifsr->cur_frameoffset = pi->frameoffset;
-#ifdef DEBUG
-             printf_filtered ("\tSaved register r%d, offset %d", reg, pifsr->offset);
-#endif
+             pifsr->cur_frameoffset = pi->sp_offset;
              pifsr++;
            }
        }
     }
-#ifdef DEBUG
-  printf_filtered ("\tfound ctret after regsave func");
-#endif
 
-  /* Set result parameters. */
+  /* Set result parameters.  */
   *pifsr_ptr = pifsr;
 }
-\f
 
+/* Helper function to evaluate if register is one of the "save" registers.
+   This allows to simplify conditionals in v850_analyze_prologue a lot.  */
 
+static int
+v850_is_save_register (int reg)
+{
+ /* The caller-save registers are R2, R20 - R29 and R31.  All other
+    registers are either special purpose (PC, SP), argument registers,
+    or just considered free for use in the caller. */
+ return reg == E_R2_REGNUM
+       || (reg >= E_R20_REGNUM && reg <= E_R29_REGNUM)
+       || reg == E_R31_REGNUM;
+}
 
-/* Function: scan_prologue
-   Scan the prologue of the function that contains PC, and record what
+/* Scan the prologue of the function that contains PC, and record what
    we find in PI.  Returns the pc after the prologue.  Note that the
    addresses saved in frame->saved_regs are just frame relative (negative
    offsets from the frame pointer).  This is because we don't know the
@@ -546,9 +455,11 @@ handle_pushm (int insn, int insn2, struct prologue_info *pi,
    prologue.  */
 
 static CORE_ADDR
-v850_scan_prologue (CORE_ADDR pc, struct prologue_info *pi)
+v850_analyze_prologue (CORE_ADDR func_addr, CORE_ADDR pc,
+                      struct v850_frame_cache *pi, ULONGEST ctbp)
 {
-  CORE_ADDR func_addr, prologue_end, current_pc;
+  CORE_ADDR prologue_end, current_pc;
+  struct pifsr pifsrs[E_NUM_REGS + 1];
   struct pifsr *pifsr, *pifsr_tmp;
   int fp_used;
   int ep_used;
@@ -557,71 +468,33 @@ v850_scan_prologue (CORE_ADDR pc, struct prologue_info *pi)
   int regsave_func_p;
   int r12_tmp;
 
-  /* First, figure out the bounds of the prologue so that we can limit the
-     search to something reasonable.  */
+  memset (&pifsrs, 0, sizeof pifsrs);
+  pifsr = &pifsrs[0];
 
-  if (find_pc_partial_function (pc, NULL, &func_addr, NULL))
-    {
-      struct symtab_and_line sal;
-
-      sal = find_pc_line (func_addr, 0);
-
-      if (func_addr == entry_point_address ())
-       pi->start_function = 1;
-      else
-       pi->start_function = 0;
-
-#if 0
-      if (sal.line == 0)
-       prologue_end = pc;
-      else
-       prologue_end = sal.end;
-#else
-      prologue_end = pc;
-#endif
-    }
-  else
-    {                          /* We're in the boondocks */
-      func_addr = pc - 100;
-      prologue_end = pc;
-    }
-
-  prologue_end = min (prologue_end, pc);
+  prologue_end = pc;
 
   /* Now, search the prologue looking for instructions that setup fp, save
      rp, adjust sp and such.  We also record the frame offset of any saved
-     registers. */
+     registers.  */
 
-  pi->frameoffset = 0;
-  pi->framereg = E_SP_REGNUM;
-  fp_used = 0;
+  pi->sp_offset = 0;
+  pi->uses_fp = 0;
   ep_used = 0;
-  pifsr = pi->pifsrs;
   regsave_func_p = 0;
   save_pc = 0;
   save_end = 0;
   r12_tmp = 0;
 
-#ifdef DEBUG
-  printf_filtered ("Current_pc = 0x%.8lx, prologue_end = 0x%.8lx\n",
-                  (long) func_addr, (long) prologue_end);
-#endif
-
   for (current_pc = func_addr; current_pc < prologue_end;)
     {
       int insn;
       int insn2 = -1; /* dummy value */
 
-#ifdef DEBUG
-      printf_filtered ("0x%.8lx ", (long) current_pc);
-      TARGET_PRINT_INSN (current_pc, &tm_print_insn_info);
-#endif
-
-      insn = read_memory_unsigned_integer (current_pc, 2);
+      insn = read_memory_integer (current_pc, 2);
       current_pc += 2;
       if ((insn & 0x0780) >= 0x0600)   /* Four byte instruction? */
        {
-         insn2 = read_memory_unsigned_integer (current_pc, 2);
+         insn2 = read_memory_integer (current_pc, 2);
          current_pc += 2;
        }
 
@@ -637,20 +510,13 @@ v850_scan_prologue (CORE_ADDR pc, struct prologue_info *pi)
          current_pc += disp - 4;
          prologue_end = (current_pc
                          + (2 * 3)     /* moves to/from ep */
-                         + 4   /* addi <const>,sp,sp */
-                         + 2   /* jmp [r10] */
+                         + 4           /* addi <const>,sp,sp */
+                         + 2           /* jmp [r10] */
                          + (2 * 12)    /* sst.w to save r2, r20-r29, r31 */
                          + 20);        /* slop area */
-
-#ifdef DEBUG
-         printf_filtered ("\tfound jarl <func>,r10, disp = %ld, low_disp = %ld, new pc = 0x%.8lx\n",
-                          disp, low_disp, (long) current_pc + 2);
-#endif
-         continue;
        }
       else if ((insn & 0xffc0) == 0x0200 && !regsave_func_p)
        {                       /* callt <imm6> */
-         long ctbp = read_register (E_CTBP_REGNUM);
          long adr = ctbp + ((insn & 0x3f) << 1);
 
          save_pc = current_pc;
@@ -659,198 +525,104 @@ v850_scan_prologue (CORE_ADDR pc, struct prologue_info *pi)
          current_pc = ctbp + (read_memory_unsigned_integer (adr, 2) & 0xffff);
          prologue_end = (current_pc
                          + (2 * 3)     /* prepare list2,imm5,sp/imm */
-                         + 4   /* ctret */
+                         + 4           /* ctret */
                          + 20);        /* slop area */
-
-#ifdef DEBUG
-         printf_filtered ("\tfound callt,  ctbp = 0x%.8lx, adr = %.8lx, new pc = 0x%.8lx\n",
-                          ctbp, adr, (long) current_pc);
-#endif
          continue;
        }
       else if ((insn & 0xffc0) == 0x0780)      /* prepare list2,imm5 */
        {
-         handle_prepare (insn, insn2, &current_pc, pi, &pifsr);
+         v850_handle_prepare (insn, insn2, &current_pc, pi, &pifsr);
          continue;
        }
       else if (insn == 0x07e0 && regsave_func_p && insn2 == 0x0144)
-       {                       /* ctret after processing register save function */
+       {                       /* ctret after processing register save */
          current_pc = save_pc;
          prologue_end = save_end;
          regsave_func_p = 0;
-#ifdef DEBUG
-         printf_filtered ("\tfound ctret after regsave func");
-#endif
          continue;
        }
       else if ((insn & 0xfff0) == 0x07e0 && (insn2 & 5) == 1)
        {                       /* pushml, pushmh */
-         handle_pushm (insn, insn2, pi, &pifsr);
+         v850_handle_pushm (insn, insn2, pi, &pifsr);
          continue;
        }
       else if ((insn & 0xffe0) == 0x0060 && regsave_func_p)
-       {                       /* jmp after processing register save function */
+       {                       /* jmp after processing register save */
          current_pc = save_pc;
          prologue_end = save_end;
          regsave_func_p = 0;
-#ifdef DEBUG
-         printf_filtered ("\tfound jmp after regsave func");
-#endif
          continue;
        }
       else if ((insn & 0x07c0) == 0x0780       /* jarl or jr */
               || (insn & 0xffe0) == 0x0060     /* jmp */
               || (insn & 0x0780) == 0x0580)    /* branch */
        {
-#ifdef DEBUG
-         printf_filtered ("\n");
-#endif
          break;                /* Ran into end of prologue */
        }
 
-      else if ((insn & 0xffe0) == ((E_SP_REGNUM << 11) | 0x0240))              /* add <imm>,sp */
-       pi->frameoffset += ((insn & 0x1f) ^ 0x10) - 0x10;
-      else if (insn == ((E_SP_REGNUM << 11) | 0x0600 | E_SP_REGNUM))   /* addi <imm>,sp,sp */
-       pi->frameoffset += insn2;
-      else if (insn == ((E_FP_RAW_REGNUM << 11) | 0x0000 | E_SP_REGNUM))       /* mov sp,fp */
-       {
-         fp_used = 1;
-         pi->framereg = E_FP_RAW_REGNUM;
-       }
-
-      else if (insn == ((E_R12_REGNUM << 11) | 0x0640 | E_R0_REGNUM))  /* movhi hi(const),r0,r12 */
+      else if ((insn & 0xffe0) == ((E_SP_REGNUM << 11) | 0x0240))
+        /* add <imm>,sp */
+       pi->sp_offset += ((insn & 0x1f) ^ 0x10) - 0x10;
+      else if (insn == ((E_SP_REGNUM << 11) | 0x0600 | E_SP_REGNUM))
+        /* addi <imm>,sp,sp */
+       pi->sp_offset += insn2;
+      else if (insn == ((E_FP_REGNUM << 11) | 0x0000 | E_SP_REGNUM))
+        /* mov sp,fp */
+       pi->uses_fp = 1;
+      else if (insn == ((E_R12_REGNUM << 11) | 0x0640 | E_R0_REGNUM))
+        /* movhi hi(const),r0,r12 */
        r12_tmp = insn2 << 16;
-      else if (insn == ((E_R12_REGNUM << 11) | 0x0620 | E_R12_REGNUM)) /* movea lo(const),r12,r12 */
+      else if (insn == ((E_R12_REGNUM << 11) | 0x0620 | E_R12_REGNUM))
+        /* movea lo(const),r12,r12 */
        r12_tmp += insn2;
-      else if (insn == ((E_SP_REGNUM << 11) | 0x01c0 | E_R12_REGNUM) && r12_tmp)       /* add r12,sp */
-       pi->frameoffset = r12_tmp;
-      else if (insn == ((E_EP_REGNUM << 11) | 0x0000 | E_SP_REGNUM))   /* mov sp,ep */
+      else if (insn == ((E_SP_REGNUM << 11) | 0x01c0 | E_R12_REGNUM) && r12_tmp)
+        /* add r12,sp */
+       pi->sp_offset += r12_tmp;
+      else if (insn == ((E_EP_REGNUM << 11) | 0x0000 | E_SP_REGNUM))
+        /* mov sp,ep */
        ep_used = 1;
-      else if (insn == ((E_EP_REGNUM << 11) | 0x0000 | E_R1_REGNUM))   /* mov r1,ep */
+      else if (insn == ((E_EP_REGNUM << 11) | 0x0000 | E_R1_REGNUM))
+        /* mov r1,ep */
        ep_used = 0;
-      else if (((insn & 0x07ff) == (0x0760 | E_SP_REGNUM)              /* st.w <reg>,<offset>[sp] */
-               || (fp_used
-                   && (insn & 0x07ff) == (0x0760 | E_FP_RAW_REGNUM)))  /* st.w <reg>,<offset>[fp] */
+      else if (((insn & 0x07ff) == (0x0760 | E_SP_REGNUM)      
+               || (pi->uses_fp
+                   && (insn & 0x07ff) == (0x0760 | E_FP_REGNUM)))
               && pifsr
-              && (((reg = (insn >> 11) & 0x1f) >= E_SAVE1_START_REGNUM && reg <= E_SAVE1_END_REGNUM)
-                  || (reg >= E_SAVE2_START_REGNUM && reg <= E_SAVE2_END_REGNUM)
-                || (reg >= E_SAVE3_START_REGNUM && reg <= E_SAVE3_END_REGNUM)))
+              && v850_is_save_register (reg = (insn >> 11) & 0x1f))
        {
+         /* st.w <reg>,<offset>[sp] or st.w <reg>,<offset>[fp] */
          pifsr->reg = reg;
          pifsr->offset = insn2 & ~1;
-         pifsr->cur_frameoffset = pi->frameoffset;
-#ifdef DEBUG
-         printf_filtered ("\tSaved register r%d, offset %d", reg, pifsr->offset);
-#endif
+         pifsr->cur_frameoffset = pi->sp_offset;
          pifsr++;
        }
-
-      else if (ep_used         /* sst.w <reg>,<offset>[ep] */
+      else if (ep_used
               && ((insn & 0x0781) == 0x0501)
               && pifsr
-              && (((reg = (insn >> 11) & 0x1f) >= E_SAVE1_START_REGNUM && reg <= E_SAVE1_END_REGNUM)
-                  || (reg >= E_SAVE2_START_REGNUM && reg <= E_SAVE2_END_REGNUM)
-                || (reg >= E_SAVE3_START_REGNUM && reg <= E_SAVE3_END_REGNUM)))
+              && v850_is_save_register (reg = (insn >> 11) & 0x1f))
        {
+         /* sst.w <reg>,<offset>[ep] */
          pifsr->reg = reg;
          pifsr->offset = (insn & 0x007e) << 1;
-         pifsr->cur_frameoffset = pi->frameoffset;
-#ifdef DEBUG
-         printf_filtered ("\tSaved register r%d, offset %d", reg, pifsr->offset);
-#endif
+         pifsr->cur_frameoffset = pi->sp_offset;
          pifsr++;
        }
-
-#ifdef DEBUG
-      printf_filtered ("\n");
-#endif
     }
 
-  if (pifsr)
-    pifsr->framereg = 0;       /* Tie off last entry */
-
-  /* Fix up any offsets to the final offset.  If a frame pointer was created, use it
-     instead of the stack pointer.  */
-  for (pifsr_tmp = pi->pifsrs; pifsr_tmp && pifsr_tmp != pifsr; pifsr_tmp++)
+  /* Fix up any offsets to the final offset.  If a frame pointer was created,
+     use it instead of the stack pointer.  */
+  for (pifsr_tmp = pifsrs; pifsr_tmp != pifsr; pifsr_tmp++)
     {
-      pifsr_tmp->offset -= pi->frameoffset - pifsr_tmp->cur_frameoffset;
-      pifsr_tmp->framereg = pi->framereg;
-
-#ifdef DEBUG
-      printf_filtered ("Saved register r%d, offset = %d, framereg = r%d\n",
-                   pifsr_tmp->reg, pifsr_tmp->offset, pifsr_tmp->framereg);
-#endif
+      pifsr_tmp->offset -= pi->sp_offset - pifsr_tmp->cur_frameoffset;
+      pi->saved_regs[pifsr_tmp->reg].addr = pifsr_tmp->offset;
     }
 
-#ifdef DEBUG
-  printf_filtered ("Framereg = r%d, frameoffset = %d\n", pi->framereg, pi->frameoffset);
-#endif
-
   return current_pc;
 }
 
-/* Function: find_callers_reg
-   Find REGNUM on the stack.  Otherwise, it's in an active register.
-   One thing we might want to do here is to check REGNUM against the
-   clobber mask, and somehow flag it as invalid if it isn't saved on
-   the stack somewhere.  This would provide a graceful failure mode
-   when trying to get the value of caller-saves registers for an inner
-   frame.  */
-
-CORE_ADDR
-v850_find_callers_reg (struct frame_info *fi, int regnum)
-{
-  for (; fi; fi = get_next_frame (fi))
-    if (DEPRECATED_PC_IN_CALL_DUMMY (get_frame_pc (fi), get_frame_base (fi),
-                                    get_frame_base (fi)))
-      return deprecated_read_register_dummy (get_frame_pc (fi),
-                                            get_frame_base (fi), regnum);
-    else if (get_frame_saved_regs (fi)[regnum] != 0)
-      return read_memory_unsigned_integer (get_frame_saved_regs (fi)[regnum],
-                                          v850_register_raw_size (regnum));
-
-  return read_register (regnum);
-}
-
-/* Function: frame_chain
-   Figure out the frame prior to FI.  Unfortunately, this involves
-   scanning the prologue of the caller, which will also be done
-   shortly by v850_init_extra_frame_info.  For the dummy frame, we
-   just return the stack pointer that was in use at the time the
-   function call was made.  */
+/* Return the address of the first code past the prologue of the function.  */
 
-CORE_ADDR
-v850_frame_chain (struct frame_info *fi)
-{
-  struct prologue_info pi;
-  CORE_ADDR callers_pc, fp;
-
-  /* First, find out who called us */
-  callers_pc = FRAME_SAVED_PC (fi);
-  /* If caller is a call-dummy, then our FP bears no relation to his FP! */
-  fp = v850_find_callers_reg (fi, E_FP_RAW_REGNUM);
-  if (DEPRECATED_PC_IN_CALL_DUMMY (callers_pc, fp, fp))
-    return fp;                 /* caller is call-dummy: return oldest value of FP */
-
-  /* Caller is NOT a call-dummy, so everything else should just work.
-     Even if THIS frame is a call-dummy! */
-  pi.pifsrs = NULL;
-
-  v850_scan_prologue (callers_pc, &pi);
-
-  if (pi.start_function)
-    return 0;                  /* Don't chain beyond the start function */
-
-  if (pi.framereg == E_FP_RAW_REGNUM)
-    return v850_find_callers_reg (fi, pi.framereg);
-
-  return get_frame_base (fi) - pi.frameoffset;
-}
-
-/* Function: skip_prologue
-   Return the address of the first code past the prologue of the function.  */
-
-CORE_ADDR
+static CORE_ADDR
 v850_skip_prologue (CORE_ADDR pc)
 {
   CORE_ADDR func_addr, func_end;
@@ -862,89 +634,66 @@ v850_skip_prologue (CORE_ADDR pc)
       struct symtab_and_line sal;
 
       sal = find_pc_line (func_addr, 0);
-
       if (sal.line != 0 && sal.end < func_end)
        return sal.end;
-      else
-       /* Either there's no line info, or the line after the prologue is after
-          the end of the function.  In this case, there probably isn't a
-          prologue.  */
-       return pc;
+
+      /* Either there's no line info, or the line after the prologue is after
+        the end of the function.  In this case, there probably isn't a
+        prologue.  */
+      return pc;
     }
 
-/* We can't find the start of this function, so there's nothing we can do. */
+  /* We can't find the start of this function, so there's nothing we can do.  */
   return pc;
 }
 
-/* Function: pop_frame
-   This routine gets called when either the user uses the `return'
-   command, or the call dummy breakpoint gets hit.  */
-
-void
-v850_pop_frame (void)
+static CORE_ADDR
+v850_frame_align (struct gdbarch *ignore, CORE_ADDR sp)
 {
-  struct frame_info *frame = get_current_frame ();
-  int regnum;
-
-  if (DEPRECATED_PC_IN_CALL_DUMMY (get_frame_pc (frame),
-                                  get_frame_base (frame),
-                                  get_frame_base (frame)))
-    generic_pop_dummy_frame ();
-  else
-    {
-      write_register (E_PC_REGNUM, FRAME_SAVED_PC (frame));
-
-      for (regnum = 0; regnum < E_NUM_REGS; regnum++)
-       if (get_frame_saved_regs (frame)[regnum] != 0)
-         write_register (regnum,
-                     read_memory_unsigned_integer (get_frame_saved_regs (frame)[regnum],
-                                            v850_register_raw_size (regnum)));
-
-      write_register (E_SP_REGNUM, get_frame_base (frame));
-    }
-
-  flush_cached_frames ();
+  return sp & ~3;
 }
 
-/* Function: push_arguments
-   Setup arguments and RP for a call to the target.  First four args
+/* Setup arguments and LP for a call to the target.  First four args
    go in R6->R9, subsequent args go into sp + 16 -> sp + ...  Structs
-   are passed by reference.  64 bit quantities (doubles and long
-   longs) may be split between the regs and the stack.  When calling a
-   function that returns a struct, a pointer to the struct is passed
-   in as a secret first argument (always in R6).
+   are passed by reference.  64 bit quantities (doubles and long longs)
+   may be split between the regs and the stack.  When calling a function
+   that returns a struct, a pointer to the struct is passed in as a secret
+   first argument (always in R6).
 
-   Stack space for the args has NOT been allocated: that job is up to us.
- */
+   Stack space for the args has NOT been allocated: that job is up to us.  */
 
-CORE_ADDR
-v850_push_arguments (int nargs, struct value **args, CORE_ADDR sp,
-                    int struct_return, CORE_ADDR struct_addr)
+static CORE_ADDR
+v850_push_dummy_call (struct gdbarch *gdbarch,
+                     struct value *function,
+                     struct regcache *regcache,
+                     CORE_ADDR bp_addr,
+                     int nargs,
+                     struct value **args,
+                     CORE_ADDR sp,
+                     int struct_return,
+                     CORE_ADDR struct_addr)
 {
   int argreg;
   int argnum;
   int len = 0;
   int stack_offset;
 
-  /* First, just for safety, make sure stack is aligned */
-  sp &= ~3;
-
   /* The offset onto the stack at which we will start copying parameters
      (after the registers are used up) begins at 16 rather than at zero.
-     I don't really know why, that's just the way it seems to work.  */
+     That's how the ABI is defined, though there's no indication that these
+     16 bytes are used for anything, not even for saving incoming
+     argument registers.  */
   stack_offset = 16;
 
-  /* Now make space on the stack for the args. */
+  /* Now make space on the stack for the args.  */
   for (argnum = 0; argnum < nargs; argnum++)
-    len += ((TYPE_LENGTH (VALUE_TYPE (args[argnum])) + 3) & ~3);
-  sp -= len + stack_offset;    /* possibly over-allocating, but it works... */
-  /* (you might think we could allocate 16 bytes */
-  /* less, but the ABI seems to use it all! )  */
+    len += ((TYPE_LENGTH (value_type (args[argnum])) + 3) & ~3);
+  sp -= len + stack_offset;
 
   argreg = E_ARG0_REGNUM;
-  /* the struct_return pointer occupies the first parameter-passing reg */
+  /* The struct_return pointer occupies the first parameter register.  */
   if (struct_return)
-    argreg++;
+    regcache_cooked_write_unsigned (regcache, argreg++, struct_addr);
 
   /* Now load as many as possible of the first arguments into
      registers, and push the rest onto the stack.  There are 16 bytes
@@ -952,20 +701,20 @@ v850_push_arguments (int nargs, struct value **args, CORE_ADDR sp,
   for (argnum = 0; argnum < nargs; argnum++)
     {
       int len;
-      char *val;
-      char valbuf[v850_register_raw_size (E_ARG0_REGNUM)];
+      gdb_byte *val;
+      gdb_byte valbuf[v850_reg_size];
 
-      if (!v850_type_is_scalar (VALUE_TYPE (*args))
-         && TYPE_LENGTH (VALUE_TYPE (*args)) > E_MAX_RETTYPE_SIZE_IN_REGS)
+      if (!v850_type_is_scalar (value_type (*args))
+         && TYPE_LENGTH (value_type (*args)) > E_MAX_RETTYPE_SIZE_IN_REGS)
        {
-         store_address (valbuf, 4, VALUE_ADDRESS (*args));
+         store_unsigned_integer (valbuf, 4, VALUE_ADDRESS (*args));
          len = 4;
          val = valbuf;
        }
       else
        {
-         len = TYPE_LENGTH (VALUE_TYPE (*args));
-         val = (char *) VALUE_CONTENTS (*args);
+         len = TYPE_LENGTH (value_type (*args));
+         val = (gdb_byte *) value_contents (*args);
        }
 
       while (len > 0)
@@ -973,11 +722,11 @@ v850_push_arguments (int nargs, struct value **args, CORE_ADDR sp,
          {
            CORE_ADDR regval;
 
-           regval = extract_address (val, v850_register_raw_size (argreg));
-           write_register (argreg, regval);
+           regval = extract_unsigned_integer (val, v850_reg_size);
+           regcache_cooked_write_unsigned (regcache, argreg, regval);
 
-           len -= v850_register_raw_size (argreg);
-           val += v850_register_raw_size (argreg);
+           len -= v850_reg_size;
+           val += v850_reg_size;
            argreg++;
          }
        else
@@ -990,89 +739,70 @@ v850_push_arguments (int nargs, struct value **args, CORE_ADDR sp,
          }
       args++;
     }
-  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 */
+  /* Store return address.  */
+  regcache_cooked_write_unsigned (regcache, E_LP_REGNUM, bp_addr);
 
-CORE_ADDR
-v850_push_return_address (CORE_ADDR pc, CORE_ADDR sp)
-{
-  write_register (E_RP_REGNUM, CALL_DUMMY_ADDRESS ());
-  return sp;
-}
+  /* Update stack pointer.  */
+  regcache_cooked_write_unsigned (regcache, E_SP_REGNUM, sp);
 
-/* Function: frame_saved_pc 
-   Find the caller of this frame.  We do this by seeing if E_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.  */
-
-CORE_ADDR
-v850_frame_saved_pc (struct frame_info *fi)
-{
-  if (DEPRECATED_PC_IN_CALL_DUMMY (get_frame_pc (fi), get_frame_base (fi),
-                                  get_frame_base (fi)))
-    return deprecated_read_register_dummy (get_frame_pc (fi),
-                                          get_frame_base (fi), E_PC_REGNUM);
-  else
-    return v850_find_callers_reg (fi, E_RP_REGNUM);
+  return sp;
 }
 
-
-/* Function: fix_call_dummy
-   Pokes the callee function's address into the CALL_DUMMY assembly stub.
-   Assumes that the CALL_DUMMY looks like this:
-   jarl <offset24>, r31
-   trap
- */
-
-void
-v850_fix_call_dummy (char *dummy, CORE_ADDR sp, CORE_ADDR fun, int nargs,
-                    struct value **args, struct type *type, int gcc_p)
+static void
+v850_extract_return_value (struct type *type, struct regcache *regcache,
+                          gdb_byte *valbuf)
 {
-  long offset24;
-
-  offset24 = (long) fun - (long) entry_point_address ();
-  offset24 &= 0x3fffff;
-  offset24 |= 0xff800000;      /* jarl <offset24>, r31 */
+  int len = TYPE_LENGTH (type);
 
-  store_unsigned_integer ((unsigned int *) &dummy[2], 2, offset24 & 0xffff);
-  store_unsigned_integer ((unsigned int *) &dummy[0], 2, offset24 >> 16);
-}
+  if (len <= v850_reg_size)
+    {
+      ULONGEST val;
 
-static CORE_ADDR
-v850_saved_pc_after_call (struct frame_info *ignore)
-{
-  return read_register (E_RP_REGNUM);
+      regcache_cooked_read_unsigned (regcache, E_V0_REGNUM, &val);
+      store_unsigned_integer (valbuf, len, val);
+    }
+  else if (len <= 2 * v850_reg_size)
+    {
+      int i, regnum = E_V0_REGNUM;
+      gdb_byte buf[v850_reg_size];
+      for (i = 0; len > 0; i += 4, len -= 4)
+       {
+         regcache_raw_read (regcache, regnum++, buf);
+         memcpy (valbuf + i, buf, len > 4 ? 4 : len);
+       }
+    }
 }
 
 static void
-v850_extract_return_value (struct type *type, char *regbuf, char *valbuf)
+v850_store_return_value (struct type *type, struct regcache *regcache,
+                        const gdb_byte *valbuf)
 {
-  CORE_ADDR return_buffer;
+  int len = TYPE_LENGTH (type);
 
-  if (!v850_use_struct_convention (0, type))
+  if (len <= v850_reg_size)
+      regcache_cooked_write_unsigned (regcache, E_V0_REGNUM,
+                                     extract_unsigned_integer (valbuf, len));
+  else if (len <= 2 * v850_reg_size)
     {
-      /* Scalar return values of <= 8 bytes are returned in 
-         E_V0_REGNUM to E_V1_REGNUM. */
-      memcpy (valbuf,
-             &regbuf[REGISTER_BYTE (E_V0_REGNUM)],
-             TYPE_LENGTH (type));
+      int i, regnum = E_V0_REGNUM;
+      for (i = 0; i < len; i += 4)
+       regcache_raw_write (regcache, regnum++, valbuf + i);
     }
-  else
-    {
-      /* Aggregates and return values > 8 bytes are returned in memory,
-         pointed to by R6. */
-      return_buffer =
-       extract_address (regbuf + REGISTER_BYTE (E_V0_REGNUM),
-                        REGISTER_RAW_SIZE (E_V0_REGNUM));
+}
 
-      read_memory (return_buffer, valbuf, TYPE_LENGTH (type));
-    }
+static enum return_value_convention
+v850_return_value (struct gdbarch *gdbarch, struct type *type,
+                  struct regcache *regcache,
+                  gdb_byte *readbuf, const gdb_byte *writebuf)
+{
+  if (v850_use_struct_convention (type))
+    return RETURN_VALUE_STRUCT_CONVENTION;
+  if (writebuf)
+    v850_store_return_value (type, regcache, writebuf);
+  else if (readbuf)
+    v850_extract_return_value (type, regcache, readbuf);
+  return RETURN_VALUE_REGISTER_CONVENTION;
 }
 
 const static unsigned char *
@@ -1083,226 +813,235 @@ v850_breakpoint_from_pc (CORE_ADDR *pcptr, int *lenptr)
   return breakpoint;
 }
 
-static CORE_ADDR
-v850_extract_struct_value_address (char *regbuf)
+static struct v850_frame_cache *
+v850_alloc_frame_cache (struct frame_info *next_frame)
 {
-  return extract_address (regbuf + v850_register_byte (E_V0_REGNUM),
-                         v850_register_raw_size (E_V0_REGNUM));
-}
+  struct v850_frame_cache *cache;
+  int i;
 
-static void
-v850_store_return_value (struct type *type, char *valbuf)
-{
-  CORE_ADDR return_buffer;
+  cache = FRAME_OBSTACK_ZALLOC (struct v850_frame_cache);
+  cache->saved_regs = trad_frame_alloc_saved_regs (next_frame);
 
-  if (!v850_use_struct_convention (0, type))
-    deprecated_write_register_bytes (REGISTER_BYTE (E_V0_REGNUM), valbuf,
-                                    TYPE_LENGTH (type));
-  else
-    {
-      return_buffer = read_register (E_V0_REGNUM);
-      write_memory (return_buffer, valbuf, TYPE_LENGTH (type));
-    }
+  /* Base address.  */
+  cache->base = 0;
+  cache->sp_offset = 0;
+  cache->pc = 0;
+
+  /* Frameless until proven otherwise.  */
+  cache->uses_fp = 0;
+
+  return cache;
 }
 
-static void
-v850_frame_init_saved_regs (struct frame_info *fi)
+static struct v850_frame_cache *
+v850_frame_cache (struct frame_info *next_frame, void **this_cache)
 {
-  struct prologue_info pi;
-  struct pifsr pifsrs[E_NUM_REGS + 1], *pifsr;
-  CORE_ADDR func_addr, func_end;
+  struct v850_frame_cache *cache;
+  CORE_ADDR current_pc;
+  int i;
+
+  if (*this_cache)
+    return *this_cache;
+
+  cache = v850_alloc_frame_cache (next_frame);
+  *this_cache = cache;
+
+  /* In principle, for normal frames, fp holds the frame pointer,
+     which holds the base address for the current stack frame.
+     However, for functions that don't need it, the frame pointer is
+     optional.  For these "frameless" functions the frame pointer is
+     actually the frame pointer of the calling frame.  */
+  cache->base = frame_unwind_register_unsigned (next_frame, E_FP_REGNUM);
+  if (cache->base == 0)
+    return cache;
 
-  if (!get_frame_saved_regs (fi))
+  cache->pc = frame_func_unwind (next_frame, NORMAL_FRAME);
+  current_pc = frame_pc_unwind (next_frame);
+  if (cache->pc != 0)
     {
-      frame_saved_regs_zalloc (fi);
+      ULONGEST ctbp;
+      ctbp = frame_unwind_register_unsigned (next_frame, E_CTBP_REGNUM);
+      v850_analyze_prologue (cache->pc, current_pc, cache, ctbp);
+    }
 
-      /* The call dummy doesn't save any registers on the stack, so we
-         can return now.  */
-      if (DEPRECATED_PC_IN_CALL_DUMMY (get_frame_pc (fi), get_frame_base (fi),
-                                      get_frame_base (fi)))
-       return;
+  if (!cache->uses_fp)
+    {
+      /* We didn't find a valid frame, which means that CACHE->base
+         currently holds the frame pointer for our calling frame.  If
+         we're at the start of a function, or somewhere half-way its
+         prologue, the function's frame probably hasn't been fully
+         setup yet.  Try to reconstruct the base address for the stack
+         frame by looking at the stack pointer.  For truly "frameless"
+         functions this might work too.  */
+      cache->base = frame_unwind_register_unsigned (next_frame, E_SP_REGNUM);
+    }
 
-      /* Find the beginning of this function, so we can analyze its
-         prologue. */
-      if (find_pc_partial_function (get_frame_pc (fi), NULL, &func_addr, &func_end))
-       {
-         pi.pifsrs = pifsrs;
+  /* Now that we have the base address for the stack frame we can
+     calculate the value of sp in the calling frame.  */
+  trad_frame_set_value (cache->saved_regs, E_SP_REGNUM,
+                       cache->base - cache->sp_offset);
 
-         v850_scan_prologue (get_frame_pc (fi), &pi);
+  /* Adjust all the saved registers such that they contain addresses
+     instead of offsets.  */
+  for (i = 0; i < E_NUM_REGS; i++)
+    if (trad_frame_addr_p (cache->saved_regs, i))
+      cache->saved_regs[i].addr += cache->base;
 
-         if (!get_next_frame (fi) && pi.framereg == E_SP_REGNUM)
-           deprecated_update_frame_base_hack (fi, read_register (pi.framereg) - pi.frameoffset);
+  /* The call instruction moves the caller's PC in the callee's LP.
+     Since this is an unwind, do the reverse.  Copy the location of LP
+     into PC (the address / regnum) so that a request for PC will be
+     converted into a request for the LP.  */
 
-         for (pifsr = pifsrs; pifsr->framereg; pifsr++)
-           {
-             get_frame_saved_regs (fi)[pifsr->reg] = pifsr->offset + get_frame_base (fi);
+  cache->saved_regs[E_PC_REGNUM] = cache->saved_regs[E_LP_REGNUM];
 
-             if (pifsr->framereg == E_SP_REGNUM)
-               get_frame_saved_regs (fi)[pifsr->reg] += pi.frameoffset;
-           }
-       }
-      /* Else we're out of luck (can't debug completely stripped code). 
-         FIXME. */
-    }
+  return cache;
 }
 
-/* Function: init_extra_frame_info
-   Setup the frame's frame pointer, pc, and frame addresses for saved
-   registers.  Most of the work is done in scan_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 (since that is derived from v850_frame_chain ()).
+static void
+v850_frame_prev_register (struct frame_info *next_frame, void **this_cache,
+                         int regnum, int *optimizedp,
+                         enum lval_type *lvalp, CORE_ADDR *addrp,
+                         int *realnump, gdb_byte *valuep)
+{
+  struct v850_frame_cache *cache = v850_frame_cache (next_frame, this_cache);
 
-   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 run_stack_dummy).  */
+  gdb_assert (regnum >= 0);
+
+  trad_frame_get_prev_register (next_frame, cache->saved_regs, regnum,
+                               optimizedp, lvalp, addrp, realnump, valuep);
+}
 
 static void
-v850_init_extra_frame_info (int fromleaf, struct frame_info *fi)
+v850_frame_this_id (struct frame_info *next_frame, void **this_cache,
+                   struct frame_id *this_id)
 {
-  struct prologue_info pi;
+  struct v850_frame_cache *cache = v850_frame_cache (next_frame, this_cache);
 
-  if (get_next_frame (fi))
-    deprecated_update_frame_pc_hack (fi, FRAME_SAVED_PC (get_next_frame (fi)));
+  /* This marks the outermost frame.  */
+  if (cache->base == 0)
+    return;
 
-  v850_frame_init_saved_regs (fi);
+  *this_id = frame_id_build (cache->saved_regs[E_SP_REGNUM].addr, cache->pc);
 }
 
-static void
-v850_store_struct_return (CORE_ADDR addr, CORE_ADDR sp)
+static const struct frame_unwind v850_frame_unwind = {
+  NORMAL_FRAME,
+  v850_frame_this_id,
+  v850_frame_prev_register
+};
+    
+static const struct frame_unwind *
+v850_frame_sniffer (struct frame_info *next_frame)
+{     
+  return &v850_frame_unwind;
+}
+
+static CORE_ADDR
+v850_unwind_sp (struct gdbarch *gdbarch, struct frame_info *next_frame)
+{
+  return frame_unwind_register_unsigned (next_frame, SP_REGNUM);
+} 
+
+static CORE_ADDR
+v850_unwind_pc (struct gdbarch *gdbarch, struct frame_info *next_frame)
 {
-  write_register (E_ARG0_REGNUM, addr);
+  return frame_unwind_register_unsigned (next_frame, PC_REGNUM);
 }
 
+static struct frame_id
+v850_unwind_dummy_id (struct gdbarch *gdbarch, struct frame_info *next_frame)
+{
+  return frame_id_build (v850_unwind_sp (gdbarch, next_frame),
+                         frame_pc_unwind (next_frame));
+}
+  
 static CORE_ADDR
-v850_target_read_fp (void)
+v850_frame_base_address (struct frame_info *next_frame, void **this_cache)
 {
-  return read_register (E_FP_RAW_REGNUM);
+  struct v850_frame_cache *cache = v850_frame_cache (next_frame, this_cache);
+
+  return cache->base;
 }
 
+static const struct frame_base v850_frame_base = {
+  &v850_frame_unwind,
+  v850_frame_base_address,
+  v850_frame_base_address,
+  v850_frame_base_address
+};
+
 static struct gdbarch *
 v850_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
 {
-  static LONGEST call_dummy_words[1] = { 0 };
-  struct gdbarch_tdep *tdep = NULL;
   struct gdbarch *gdbarch;
-  int i;
 
-  /* find a candidate among the list of pre-declared architectures. */
-  arches = gdbarch_list_lookup_by_info (arches, &info);
-  if (arches != NULL)
-    return (arches->gdbarch);
-
-#if 0
-  tdep = (struct gdbarch_tdep *) xmalloc (sizeof (struct gdbarch_tdep));
-#endif
-
-  /* Change the register names based on the current machine type. */
+  /* Change the register names based on the current machine type.  */
   if (info.bfd_arch_info->arch != bfd_arch_v850)
-    return 0;
+    return NULL;
 
-  gdbarch = gdbarch_alloc (&info, 0);
+  gdbarch = gdbarch_alloc (&info, NULL);
 
-  /* NOTE: cagney/2002-12-06: This can be deleted when this arch is
-     ready to unwind the PC first (see frame.c:get_prev_frame()).  */
-  set_gdbarch_deprecated_init_frame_pc (gdbarch, init_frame_pc_default);
-
-  for (i = 0; v850_processor_type_table[i].regnames != NULL; i++)
+  switch (info.bfd_arch_info->mach)
     {
-      if (v850_processor_type_table[i].mach == info.bfd_arch_info->mach)
-       {
-         v850_register_names = v850_processor_type_table[i].regnames;
-         tm_print_insn_info.mach = info.bfd_arch_info->mach;
-         break;
-       }
+    case bfd_mach_v850:
+      set_gdbarch_register_name (gdbarch, v850_register_name);
+      break;
+    case bfd_mach_v850e:
+    case bfd_mach_v850e1:
+      set_gdbarch_register_name (gdbarch, v850e_register_name);
+      break;
     }
 
-  /*
-   * Basic register fields and methods.
-   */
   set_gdbarch_num_regs (gdbarch, E_NUM_REGS);
   set_gdbarch_num_pseudo_regs (gdbarch, 0);
   set_gdbarch_sp_regnum (gdbarch, E_SP_REGNUM);
-  set_gdbarch_fp_regnum (gdbarch, E_FP_REGNUM);
   set_gdbarch_pc_regnum (gdbarch, E_PC_REGNUM);
-  set_gdbarch_register_name (gdbarch, v850_register_name);
-  set_gdbarch_register_size (gdbarch, v850_reg_size);
-  set_gdbarch_register_bytes (gdbarch, E_ALL_REGS_SIZE);
-  set_gdbarch_register_byte (gdbarch, v850_register_byte);
-  set_gdbarch_register_raw_size (gdbarch, v850_register_raw_size);
-  set_gdbarch_max_register_raw_size (gdbarch, v850_reg_size);
-  set_gdbarch_register_virtual_size (gdbarch, v850_register_raw_size);
-  set_gdbarch_max_register_virtual_size (gdbarch, v850_reg_size);
-  set_gdbarch_register_virtual_type (gdbarch, v850_reg_virtual_type);
-
-  set_gdbarch_read_fp (gdbarch, v850_target_read_fp);
-
-  /*
-   * Frame Info
-   */
-  set_gdbarch_init_extra_frame_info (gdbarch, v850_init_extra_frame_info);
-  set_gdbarch_frame_init_saved_regs (gdbarch, v850_frame_init_saved_regs);
-  set_gdbarch_frame_chain (gdbarch, v850_frame_chain);
-  set_gdbarch_saved_pc_after_call (gdbarch, v850_saved_pc_after_call);
-  set_gdbarch_frame_saved_pc (gdbarch, v850_frame_saved_pc);
-  set_gdbarch_skip_prologue (gdbarch, v850_skip_prologue);
+  set_gdbarch_fp0_regnum (gdbarch, -1);
 
-  /* 
-   * Miscelany
-   */
-  /* Stack grows up. */
-  set_gdbarch_inner_than (gdbarch, core_addr_lessthan);
-  /* PC stops zero byte after a trap instruction
-     (which means: exactly on trap instruction). */
-  set_gdbarch_decr_pc_after_break (gdbarch, 0);
-  /* This value is almost never non-zero... */
-  set_gdbarch_function_start_offset (gdbarch, 0);
-  /* This value is almost never non-zero... */
-  set_gdbarch_frame_args_skip (gdbarch, 0);
-  /* OK to default this value to 'unknown'. */
-  set_gdbarch_frame_num_args (gdbarch, frame_num_args_unknown);
-
-  /*
-   * Call Dummies
-   * 
-   * These values and methods are used when gdb calls a target function.  */
-  set_gdbarch_push_dummy_frame (gdbarch, generic_push_dummy_frame);
-  set_gdbarch_push_return_address (gdbarch, v850_push_return_address);
-  set_gdbarch_deprecated_extract_return_value (gdbarch, v850_extract_return_value);
-  set_gdbarch_push_arguments (gdbarch, v850_push_arguments);
-  set_gdbarch_pop_frame (gdbarch, v850_pop_frame);
-  set_gdbarch_store_struct_return (gdbarch, v850_store_struct_return);
-  set_gdbarch_deprecated_store_return_value (gdbarch, v850_store_return_value);
-  set_gdbarch_deprecated_extract_struct_value_address (gdbarch, v850_extract_struct_value_address);
-  set_gdbarch_use_struct_convention (gdbarch, v850_use_struct_convention);
-  set_gdbarch_call_dummy_address (gdbarch, entry_point_address);
-  set_gdbarch_call_dummy_start_offset (gdbarch, 0);
-  set_gdbarch_call_dummy_breakpoint_offset (gdbarch, 0);
-  set_gdbarch_call_dummy_breakpoint_offset_p (gdbarch, 1);
-  set_gdbarch_call_dummy_length (gdbarch, 0);
-  set_gdbarch_call_dummy_p (gdbarch, 1);
-  set_gdbarch_call_dummy_words (gdbarch, call_dummy_nil);
-  set_gdbarch_sizeof_call_dummy_words (gdbarch, 0);
-  set_gdbarch_call_dummy_stack_adjust_p (gdbarch, 0);
-  /* set_gdbarch_call_dummy_stack_adjust */
-  set_gdbarch_fix_call_dummy (gdbarch, v850_fix_call_dummy);
-  set_gdbarch_breakpoint_from_pc (gdbarch, v850_breakpoint_from_pc);
+  set_gdbarch_register_type (gdbarch, v850_register_type);
 
+  set_gdbarch_char_signed (gdbarch, 0);
+  set_gdbarch_short_bit (gdbarch, 2 * TARGET_CHAR_BIT);
   set_gdbarch_int_bit (gdbarch, 4 * TARGET_CHAR_BIT);
+  set_gdbarch_long_bit (gdbarch, 4 * TARGET_CHAR_BIT);
+  set_gdbarch_long_long_bit (gdbarch, 8 * TARGET_CHAR_BIT);
+
+  set_gdbarch_float_bit (gdbarch, 4 * TARGET_CHAR_BIT);
+  set_gdbarch_double_bit (gdbarch, 8 * TARGET_CHAR_BIT);
+  set_gdbarch_long_double_bit (gdbarch, 8 * TARGET_CHAR_BIT);
+
   set_gdbarch_ptr_bit (gdbarch, 4 * TARGET_CHAR_BIT);
   set_gdbarch_addr_bit (gdbarch, 4 * TARGET_CHAR_BIT);
-  set_gdbarch_long_double_bit (gdbarch, 8 * TARGET_CHAR_BIT);
 
-  set_gdbarch_extra_stack_alignment_needed (gdbarch, 0);
+  set_gdbarch_inner_than (gdbarch, core_addr_lessthan);
+  set_gdbarch_breakpoint_from_pc (gdbarch, v850_breakpoint_from_pc);
+
+  set_gdbarch_return_value (gdbarch, v850_return_value);
+  set_gdbarch_push_dummy_call (gdbarch, v850_push_dummy_call);
+  set_gdbarch_skip_prologue (gdbarch, v850_skip_prologue);
+
+  set_gdbarch_print_insn (gdbarch, print_insn_v850);
+
+  set_gdbarch_frame_align (gdbarch, v850_frame_align);
+  set_gdbarch_unwind_sp (gdbarch, v850_unwind_sp);
+  set_gdbarch_unwind_pc (gdbarch, v850_unwind_pc);
+  set_gdbarch_unwind_dummy_id (gdbarch, v850_unwind_dummy_id);
+  frame_base_set_default (gdbarch, &v850_frame_base);
+
+  /* Hook in ABI-specific overrides, if they have been registered.  */
+  gdbarch_init_osabi (info, gdbarch);
+
+  frame_unwind_append_sniffer (gdbarch, dwarf2_frame_sniffer);
+  frame_unwind_append_sniffer (gdbarch, v850_frame_sniffer);
 
   return gdbarch;
 }
 
+extern initialize_file_ftype _initialize_v850_tdep; /* -Wmissing-prototypes */
+
 void
 _initialize_v850_tdep (void)
 {
-  tm_print_insn = print_insn_v850;
   register_gdbarch_init (bfd_arch_v850, v850_gdbarch_init);
 }
This page took 0.053652 seconds and 4 git commands to generate.