2004-02-17 Ulrich Weigand <Ulrich.Weigand@de.ibm.com>
[deliverable/binutils-gdb.git] / gdb / s390-tdep.c
index ee6574d95579eed1d64b3c727b0fb2ed712e431b..f55564e3554f5c9f523f6aa75c98b7ec4ad7812f 100644 (file)
@@ -1,6 +1,6 @@
 /* Target-dependent code for GDB, the GNU debugger.
 
-   Copyright 2001, 2002, 2003 Free Software Foundation, Inc.
+   Copyright 2001, 2002, 2003, 2004 Free Software Foundation, Inc.
 
    Contributed by D.J. Barrow (djbarrow@de.ibm.com,barrow_dj@yahoo.com)
    for IBM Deutschland Entwicklung GmbH, IBM Corporation.
@@ -22,8 +22,7 @@
    Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
    02111-1307, USA.  */
 
-#define S390_TDEP              /* for special macros in tm-s390.h */
-#include <defs.h>
+#include "defs.h"
 #include "arch-utils.h"
 #include "frame.h"
 #include "inferior.h"
 #include "target.h"
 #include "gdbcore.h"
 #include "gdbcmd.h"
-#include "symfile.h"
 #include "objfiles.h"
 #include "tm.h"
 #include "../bfd/bfd.h"
 #include "floatformat.h"
 #include "regcache.h"
+#include "reggroups.h"
+#include "regset.h"
 #include "value.h"
 #include "gdb_assert.h"
 #include "dis-asm.h"
 
+#include "s390-tdep.h"
 
 
-/* Number of bytes of storage in the actual machine representation
-   for register N.  */
-static int
-s390_register_raw_size (int reg_nr)
+/* The tdep structure.  */
+
+struct gdbarch_tdep
 {
-  if (S390_FP0_REGNUM <= reg_nr
-      && reg_nr < S390_FP0_REGNUM + S390_NUM_FPRS)
-    return S390_FPR_SIZE;
-  else
-    return 4;
+  /* ABI version.  */
+  enum { ABI_LINUX_S390, ABI_LINUX_ZSERIES } abi;
+
+  /* Core file register sets.  */
+  const struct regset *gregset;
+  int sizeof_gregset;
+
+  const struct regset *fpregset;
+  int sizeof_fpregset;
+};
+
+
+/* Register information.  */
+
+struct s390_register_info
+{
+  char *name;
+  struct type **type;
+};
+
+static struct s390_register_info s390_register_info[S390_NUM_TOTAL_REGS] = 
+{
+  /* Program Status Word.  */
+  { "pswm", &builtin_type_long },
+  { "pswa", &builtin_type_long },
+
+  /* General Purpose Registers.  */
+  { "r0", &builtin_type_long },
+  { "r1", &builtin_type_long },
+  { "r2", &builtin_type_long },
+  { "r3", &builtin_type_long },
+  { "r4", &builtin_type_long },
+  { "r5", &builtin_type_long },
+  { "r6", &builtin_type_long },
+  { "r7", &builtin_type_long },
+  { "r8", &builtin_type_long },
+  { "r9", &builtin_type_long },
+  { "r10", &builtin_type_long },
+  { "r11", &builtin_type_long },
+  { "r12", &builtin_type_long },
+  { "r13", &builtin_type_long },
+  { "r14", &builtin_type_long },
+  { "r15", &builtin_type_long },
+
+  /* Access Registers.  */
+  { "acr0", &builtin_type_int },
+  { "acr1", &builtin_type_int },
+  { "acr2", &builtin_type_int },
+  { "acr3", &builtin_type_int },
+  { "acr4", &builtin_type_int },
+  { "acr5", &builtin_type_int },
+  { "acr6", &builtin_type_int },
+  { "acr7", &builtin_type_int },
+  { "acr8", &builtin_type_int },
+  { "acr9", &builtin_type_int },
+  { "acr10", &builtin_type_int },
+  { "acr11", &builtin_type_int },
+  { "acr12", &builtin_type_int },
+  { "acr13", &builtin_type_int },
+  { "acr14", &builtin_type_int },
+  { "acr15", &builtin_type_int },
+
+  /* Floating Point Control Word.  */
+  { "fpc", &builtin_type_int },
+
+  /* Floating Point Registers.  */
+  { "f0", &builtin_type_double },
+  { "f1", &builtin_type_double },
+  { "f2", &builtin_type_double },
+  { "f3", &builtin_type_double },
+  { "f4", &builtin_type_double },
+  { "f5", &builtin_type_double },
+  { "f6", &builtin_type_double },
+  { "f7", &builtin_type_double },
+  { "f8", &builtin_type_double },
+  { "f9", &builtin_type_double },
+  { "f10", &builtin_type_double },
+  { "f11", &builtin_type_double },
+  { "f12", &builtin_type_double },
+  { "f13", &builtin_type_double },
+  { "f14", &builtin_type_double },
+  { "f15", &builtin_type_double },
+
+  /* Pseudo registers.  */
+  { "pc", &builtin_type_void_func_ptr },
+  { "cc", &builtin_type_int },
+};
+
+/* Return the name of register REGNUM.  */
+static const char *
+s390_register_name (int regnum)
+{
+  gdb_assert (regnum >= 0 && regnum < S390_NUM_TOTAL_REGS);
+  return s390_register_info[regnum].name;
+}
+
+/* Return the GDB type object for the "standard" data type of data in
+   register REGNUM. */
+static struct type *
+s390_register_type (struct gdbarch *gdbarch, int regnum)
+{
+  gdb_assert (regnum >= 0 && regnum < S390_NUM_TOTAL_REGS);
+  return *s390_register_info[regnum].type;
 }
 
+/* DWARF Register Mapping.  */
+
+static int s390_dwarf_regmap[] =
+{
+  /* General Purpose Registers.  */
+  S390_R0_REGNUM, S390_R1_REGNUM, S390_R2_REGNUM, S390_R3_REGNUM,
+  S390_R4_REGNUM, S390_R5_REGNUM, S390_R6_REGNUM, S390_R7_REGNUM,
+  S390_R8_REGNUM, S390_R9_REGNUM, S390_R10_REGNUM, S390_R11_REGNUM,
+  S390_R12_REGNUM, S390_R13_REGNUM, S390_R14_REGNUM, S390_R15_REGNUM,
+
+  /* Floating Point Registers.  */
+  S390_F0_REGNUM, S390_F2_REGNUM, S390_F4_REGNUM, S390_F6_REGNUM,
+  S390_F1_REGNUM, S390_F3_REGNUM, S390_F5_REGNUM, S390_F7_REGNUM,
+  S390_F8_REGNUM, S390_F10_REGNUM, S390_F12_REGNUM, S390_F14_REGNUM,
+  S390_F9_REGNUM, S390_F11_REGNUM, S390_F13_REGNUM, S390_F15_REGNUM,
+
+  /* Control Registers (not mapped).  */
+  -1, -1, -1, -1, -1, -1, -1, -1, 
+  -1, -1, -1, -1, -1, -1, -1, -1, 
+
+  /* Access Registers.  */
+  S390_A0_REGNUM, S390_A1_REGNUM, S390_A2_REGNUM, S390_A3_REGNUM,
+  S390_A4_REGNUM, S390_A5_REGNUM, S390_A6_REGNUM, S390_A7_REGNUM,
+  S390_A8_REGNUM, S390_A9_REGNUM, S390_A10_REGNUM, S390_A11_REGNUM,
+  S390_A12_REGNUM, S390_A13_REGNUM, S390_A14_REGNUM, S390_A15_REGNUM,
+
+  /* Program Status Word.  */
+  S390_PSWM_REGNUM,
+  S390_PSWA_REGNUM
+};
+
+/* Convert DWARF register number REG to the appropriate register
+   number used by GDB.  */
 static int
-s390x_register_raw_size (int reg_nr)
+s390_dwarf_reg_to_regnum (int reg)
+{
+  int regnum = -1;
+
+  if (reg >= 0 || reg < ARRAY_SIZE (s390_dwarf_regmap))
+    regnum = s390_dwarf_regmap[reg];
+
+  if (regnum == -1)
+    warning ("Unmapped DWARF Register #%d encountered\n", reg);
+
+  return regnum;
+}
+
+/* Pseudo registers - PC and condition code.  */
+
+static void
+s390_pseudo_register_read (struct gdbarch *gdbarch, struct regcache *regcache,
+                          int regnum, void *buf)
 {
-  return (reg_nr == S390_FPC_REGNUM)
-    || (reg_nr >= S390_FIRST_ACR && reg_nr <= S390_LAST_ACR) ? 4 : 8;
+  ULONGEST val;
+
+  switch (regnum)
+    {
+    case S390_PC_REGNUM:
+      regcache_raw_read_unsigned (regcache, S390_PSWA_REGNUM, &val);
+      store_unsigned_integer (buf, 4, val & 0x7fffffff);
+      break;
+
+    case S390_CC_REGNUM:
+      regcache_raw_read_unsigned (regcache, S390_PSWM_REGNUM, &val);
+      store_unsigned_integer (buf, 4, (val >> 12) & 3);
+      break;
+
+    default:
+      internal_error (__FILE__, __LINE__, "invalid regnum");
+    }
+}
+
+static void
+s390_pseudo_register_write (struct gdbarch *gdbarch, struct regcache *regcache,
+                           int regnum, const void *buf)
+{
+  ULONGEST val, psw;
+
+  switch (regnum)
+    {
+    case S390_PC_REGNUM:
+      val = extract_unsigned_integer (buf, 4);
+      regcache_raw_read_unsigned (regcache, S390_PSWA_REGNUM, &psw);
+      psw = (psw & 0x80000000) | (val & 0x7fffffff);
+      regcache_raw_write_unsigned (regcache, S390_PSWA_REGNUM, psw);
+      break;
+
+    case S390_CC_REGNUM:
+      val = extract_unsigned_integer (buf, 4);
+      regcache_raw_read_unsigned (regcache, S390_PSWM_REGNUM, &psw);
+      psw = (psw & ~((ULONGEST)3 << 12)) | ((val & 3) << 12);
+      regcache_raw_write_unsigned (regcache, S390_PSWM_REGNUM, psw);
+      break;
+
+    default:
+      internal_error (__FILE__, __LINE__, "invalid regnum");
+    }
+}
+
+static void
+s390x_pseudo_register_read (struct gdbarch *gdbarch, struct regcache *regcache,
+                           int regnum, void *buf)
+{
+  ULONGEST val;
+
+  switch (regnum)
+    {
+    case S390_PC_REGNUM:
+      regcache_raw_read (regcache, S390_PSWA_REGNUM, buf);
+      break;
+
+    case S390_CC_REGNUM:
+      regcache_raw_read_unsigned (regcache, S390_PSWM_REGNUM, &val);
+      store_unsigned_integer (buf, 4, (val >> 44) & 3);
+      break;
+
+    default:
+      internal_error (__FILE__, __LINE__, "invalid regnum");
+    }
+}
+
+static void
+s390x_pseudo_register_write (struct gdbarch *gdbarch, struct regcache *regcache,
+                            int regnum, const void *buf)
+{
+  ULONGEST val, psw;
+
+  switch (regnum)
+    {
+    case S390_PC_REGNUM:
+      regcache_raw_write (regcache, S390_PSWA_REGNUM, buf);
+      break;
+
+    case S390_CC_REGNUM:
+      val = extract_unsigned_integer (buf, 4);
+      regcache_raw_read_unsigned (regcache, S390_PSWM_REGNUM, &psw);
+      psw = (psw & ~((ULONGEST)3 << 44)) | ((val & 3) << 44);
+      regcache_raw_write_unsigned (regcache, S390_PSWM_REGNUM, psw);
+      break;
+
+    default:
+      internal_error (__FILE__, __LINE__, "invalid regnum");
+    }
 }
 
+/* 'float' values are stored in the upper half of floating-point
+   registers, even though we are otherwise a big-endian platform.  */
+
 static int
-s390_cannot_fetch_register (int regno)
+s390_convert_register_p (int regno, struct type *type)
+{
+  return (regno >= S390_F0_REGNUM && regno <= S390_F15_REGNUM)
+        && TYPE_LENGTH (type) < 8;
+}
+
+static void
+s390_register_to_value (struct frame_info *frame, int regnum,
+                        struct type *valtype, void *out)
+{
+  char in[8];
+  int len = TYPE_LENGTH (valtype);
+  gdb_assert (len < 8);
+
+  get_frame_register (frame, regnum, in);
+  memcpy (out, in, len);
+}
+
+static void
+s390_value_to_register (struct frame_info *frame, int regnum,
+                        struct type *valtype, const void *in)
 {
-  return (regno >= S390_FIRST_CR && regno < (S390_FIRST_CR + 9)) ||
-    (regno >= (S390_FIRST_CR + 12) && regno <= S390_LAST_CR);
+  char out[8];
+  int len = TYPE_LENGTH (valtype);
+  gdb_assert (len < 8);
+
+  memset (out, 0, 8);
+  memcpy (out, in, len);
+  put_frame_register (frame, regnum, out);
 }
 
+/* Register groups.  */
+
 static int
-s390_register_byte (int reg_nr)
-{
-  if (reg_nr <= S390_GP_LAST_REGNUM)
-    return reg_nr * S390_GPR_SIZE;
-  if (reg_nr <= S390_LAST_ACR)
-    return S390_ACR0_OFFSET + (((reg_nr) - S390_FIRST_ACR) * S390_ACR_SIZE);
-  if (reg_nr <= S390_LAST_CR)
-    return S390_CR0_OFFSET + (((reg_nr) - S390_FIRST_CR) * S390_CR_SIZE);
-  if (reg_nr == S390_FPC_REGNUM)
-    return S390_FPC_OFFSET;
-  else
-    return S390_FP0_OFFSET + (((reg_nr) - S390_FP0_REGNUM) * S390_FPR_SIZE);
+s390_register_reggroup_p (struct gdbarch *gdbarch, int regnum,
+                         struct reggroup *group)
+{
+  struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch);
+
+  /* Registers displayed via 'info regs'.  */
+  if (group == general_reggroup)
+    return (regnum >= S390_R0_REGNUM && regnum <= S390_R15_REGNUM)
+          || regnum == S390_PC_REGNUM
+          || regnum == S390_CC_REGNUM;
+
+  /* Registers displayed via 'info float'.  */
+  if (group == float_reggroup)
+    return (regnum >= S390_F0_REGNUM && regnum <= S390_F15_REGNUM)
+          || regnum == S390_FPC_REGNUM;
+
+  /* Registers that need to be saved/restored in order to
+     push or pop frames.  */
+  if (group == save_reggroup || group == restore_reggroup)
+    return regnum != S390_PSWM_REGNUM && regnum != S390_PSWA_REGNUM;
+
+  return default_register_reggroup_p (gdbarch, regnum, group);
+}
+
+
+/* Core file register sets.  */
+
+int s390_regmap_gregset[S390_NUM_REGS] =
+{
+  /* Program Status Word.  */
+  0x00, 0x04,
+  /* General Purpose Registers.  */
+  0x08, 0x0c, 0x10, 0x14,
+  0x18, 0x1c, 0x20, 0x24,
+  0x28, 0x2c, 0x30, 0x34,
+  0x38, 0x3c, 0x40, 0x44,
+  /* Access Registers.  */
+  0x48, 0x4c, 0x50, 0x54,
+  0x58, 0x5c, 0x60, 0x64,
+  0x68, 0x6c, 0x70, 0x74,
+  0x78, 0x7c, 0x80, 0x84,
+  /* Floating Point Control Word.  */
+  -1,
+  /* Floating Point Registers.  */
+  -1, -1, -1, -1, -1, -1, -1, -1,
+  -1, -1, -1, -1, -1, -1, -1, -1,
+};
+
+int s390x_regmap_gregset[S390_NUM_REGS] =
+{
+  0x00, 0x08,
+  /* General Purpose Registers.  */
+  0x10, 0x18, 0x20, 0x28,
+  0x30, 0x38, 0x40, 0x48,
+  0x50, 0x58, 0x60, 0x68,
+  0x70, 0x78, 0x80, 0x88,
+  /* Access Registers.  */
+  0x90, 0x94, 0x98, 0x9c,
+  0xa0, 0xa4, 0xa8, 0xac,
+  0xb0, 0xb4, 0xb8, 0xbc,
+  0xc0, 0xc4, 0xc8, 0xcc,
+  /* Floating Point Control Word.  */
+  -1,
+  /* Floating Point Registers.  */
+  -1, -1, -1, -1, -1, -1, -1, -1,
+  -1, -1, -1, -1, -1, -1, -1, -1,
+};
+
+int s390_regmap_fpregset[S390_NUM_REGS] =
+{
+  /* Program Status Word.  */
+  -1, -1,
+  /* General Purpose Registers.  */
+  -1, -1, -1, -1, -1, -1, -1, -1,
+  -1, -1, -1, -1, -1, -1, -1, -1,
+  /* Access Registers.  */
+  -1, -1, -1, -1, -1, -1, -1, -1,
+  -1, -1, -1, -1, -1, -1, -1, -1,
+  /* Floating Point Control Word.  */
+  0x00,
+  /* Floating Point Registers.  */
+  0x08, 0x10, 0x18, 0x20,
+  0x28, 0x30, 0x38, 0x40,
+  0x48, 0x50, 0x58, 0x60,
+  0x68, 0x70, 0x78, 0x80,
+};
+
+/* Supply register REGNUM from the register set REGSET to register cache 
+   REGCACHE.  If REGNUM is -1, do this for all registers in REGSET.  */
+static void
+s390_supply_regset (const struct regset *regset, struct regcache *regcache,
+                   int regnum, const void *regs, size_t len)
+{
+  const int *offset = regset->descr;
+  int i;
+
+  for (i = 0; i < S390_NUM_REGS; i++)
+    {
+      if ((regnum == i || regnum == -1) && offset[i] != -1)
+       regcache_raw_supply (regcache, i, (const char *)regs + offset[i]);
+    }
+}
+
+static const struct regset s390_gregset = {
+  s390_regmap_gregset, 
+  s390_supply_regset
+};
+
+static const struct regset s390x_gregset = {
+  s390x_regmap_gregset, 
+  s390_supply_regset
+};
+
+static const struct regset s390_fpregset = {
+  s390_regmap_fpregset, 
+  s390_supply_regset
+};
+
+/* Return the appropriate register set for the core section identified
+   by SECT_NAME and SECT_SIZE.  */
+const struct regset *
+s390_regset_from_core_section (struct gdbarch *gdbarch,
+                              const char *sect_name, size_t sect_size)
+{
+  struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch);
+
+  if (strcmp (sect_name, ".reg") == 0 && sect_size == tdep->sizeof_gregset)
+    return tdep->gregset;
+
+  if (strcmp (sect_name, ".reg2") == 0 && sect_size == tdep->sizeof_fpregset)
+    return tdep->fpregset;
+
+  return NULL;
 }
 
+
+#define GDB_TARGET_IS_ESAME (TARGET_ARCHITECTURE->mach == bfd_mach_s390_64)
+#define S390_GPR_SIZE      (GDB_TARGET_IS_ESAME ? 8 : 4)
+#define S390_FPR_SIZE      (8)
 #define S390_MAX_INSTR_SIZE (6)
 #define S390_SYSCALL_OPCODE (0x0a)
 #define S390_SYSCALL_SIZE   (2)
@@ -93,9 +484,6 @@ s390_register_byte (int reg_nr)
 #define S390X_SIGREGS_FP0_OFFSET      (216)
 #define S390_UC_MCONTEXT_OFFSET    (256)
 #define S390X_UC_MCONTEXT_OFFSET   (344)
-#define S390_STACK_FRAME_OVERHEAD  16*DEPRECATED_REGISTER_SIZE+32
-#define S390_STACK_PARAMETER_ALIGNMENT  DEPRECATED_REGISTER_SIZE
-#define S390_NUM_FP_PARAMETER_REGISTERS (GDB_TARGET_IS_ESAME ? 4:2)
 #define S390_SIGNAL_FRAMESIZE  (GDB_TARGET_IS_ESAME ? 160:96)
 #define s390_NR_sigreturn          119
 #define s390_NR_rt_sigreturn       173
@@ -150,44 +538,6 @@ s390_memset_extra_info (struct frame_extra_info *fextra_info)
 }
 
 
-
-static const char *
-s390_register_name (int reg_nr)
-{
-  static char *register_names[] = {
-    "pswm", "pswa",
-    "r0", "r1", "r2", "r3", "r4", "r5", "r6", "r7",
-    "r8", "r9", "r10", "r11", "r12", "r13", "r14", "r15",
-    "acr0", "acr1", "acr2", "acr3", "acr4", "acr5", "acr6", "acr7",
-    "acr8", "acr9", "acr10", "acr11", "acr12", "acr13", "acr14", "acr15",
-    "cr0", "cr1", "cr2", "cr3", "cr4", "cr5", "cr6", "cr7",
-    "cr8", "cr9", "cr10", "cr11", "cr12", "cr13", "cr14", "cr15",
-    "fpc",
-    "f0", "f1", "f2", "f3", "f4", "f5", "f6", "f7",
-    "f8", "f9", "f10", "f11", "f12", "f13", "f14", "f15"
-  };
-
-  if (reg_nr <= S390_LAST_REGNUM)
-    return register_names[reg_nr];
-  else
-    return NULL;
-}
-
-
-
-
-static int
-s390_stab_reg_to_regnum (int regno)
-{
-  return regno >= 64 ? S390_PSWM_REGNUM - 64 :
-    regno >= 48 ? S390_FIRST_ACR - 48 :
-    regno >= 32 ? S390_FIRST_CR - 32 :
-    regno <= 15 ? (regno + 2) :
-    S390_FP0_REGNUM + ((regno - 16) & 8) + (((regno - 16) & 3) << 1) +
-    (((regno - 16) & 4) >> 2);
-}
-
-
 /* Prologue analysis.  */
 
 /* When we analyze a prologue, we're really doing 'abstract
@@ -599,6 +949,8 @@ enum
     op1_stg  = 0xe3,   op2_stg  = 0x24,
     op_stm   = 0x90,
     op1_stmg = 0xeb,   op2_stmg = 0x24,
+    op_lm    = 0x98,
+    op1_lmg  = 0xeb,   op2_lmg  = 0x04,
     op_svc   = 0x0a,
   };
 
@@ -787,6 +1139,8 @@ compute_x_addr (struct prologue_value *addr,
    track general-purpose registers r2 -- r15, and floating-point
    registers f0, f2, f4, and f6.  */
 #define S390_NUM_SPILL_SLOTS (14 + 4)
+#define S390_NUM_GPRS 16
+#define S390_NUM_FPRS 16
 
 
 /* If the SIZE bytes at ADDR are a stack slot we're actually tracking,
@@ -820,7 +1174,7 @@ s390_on_stack (struct prologue_value *addr,
   /* Construct the addresses of the spill arrays and the back chain.  */
   pv_set_to_register (&gpr_spill_addr, S390_SP_REGNUM, 2 * S390_GPR_SIZE);
   pv_set_to_register (&fpr_spill_addr, S390_SP_REGNUM, 16 * S390_GPR_SIZE);
-  back_chain_addr = gpr[S390_SP_REGNUM - S390_GP0_REGNUM];
+  back_chain_addr = gpr[S390_SP_REGNUM - S390_R0_REGNUM];
 
   /* We have to check for GPR and FPR references using two separate
      calls to pv_is_array_ref, since the GPR and FPR spill slots are
@@ -901,14 +1255,14 @@ s390_get_signal_frame_info (struct frame_info *fi)
       && get_frame_extra_info (next_frame)->sigcontext)
     {
       /* We're definitely backtracing from a signal handler.  */
-      CORE_ADDR *saved_regs = get_frame_saved_regs (fi);
+      CORE_ADDR *saved_regs = deprecated_get_frame_saved_regs (fi);
       CORE_ADDR save_reg_addr = (get_frame_extra_info (next_frame)->sigcontext
-                                 + DEPRECATED_REGISTER_BYTE (S390_GP0_REGNUM));
+                                 + DEPRECATED_REGISTER_BYTE (S390_R0_REGNUM));
       int reg;
 
       for (reg = 0; reg < S390_NUM_GPRS; reg++)
         {
-          saved_regs[S390_GP0_REGNUM + reg] = save_reg_addr;
+          saved_regs[S390_R0_REGNUM + reg] = save_reg_addr;
           save_reg_addr += S390_GPR_SIZE;
         }
 
@@ -917,7 +1271,7 @@ s390_get_signal_frame_info (struct frame_info *fi)
                           S390_SIGREGS_FP0_OFFSET));
       for (reg = 0; reg < S390_NUM_FPRS; reg++)
         {
-          saved_regs[S390_FP0_REGNUM + reg] = save_reg_addr;
+          saved_regs[S390_F0_REGNUM + reg] = save_reg_addr;
           save_reg_addr += S390_FPR_SIZE;
         }
     }
@@ -967,10 +1321,10 @@ s390_get_frame_info (CORE_ADDR start_pc,
     int i;
 
     for (i = 0; i < S390_NUM_GPRS; i++)
-      pv_set_to_register (&gpr[i], S390_GP0_REGNUM + i, 0);
+      pv_set_to_register (&gpr[i], S390_R0_REGNUM + i, 0);
 
     for (i = 0; i < S390_NUM_FPRS; i++)
-      pv_set_to_register (&fpr[i], S390_FP0_REGNUM + i, 0);
+      pv_set_to_register (&fpr[i], S390_F0_REGNUM + i, 0);
 
     for (i = 0; i < S390_NUM_SPILL_SLOTS; i++)
       pv_set_to_unknown (&spill[i]);
@@ -1004,8 +1358,8 @@ s390_get_frame_info (CORE_ADDR start_pc,
 
       next_pc = pc + insn_len;
 
-      pre_insn_sp = gpr[S390_SP_REGNUM - S390_GP0_REGNUM];
-      pre_insn_fp = gpr[S390_FRAME_REGNUM - S390_GP0_REGNUM];
+      pre_insn_sp = gpr[S390_SP_REGNUM - S390_R0_REGNUM];
+      pre_insn_fp = gpr[S390_FRAME_REGNUM - S390_R0_REGNUM];
       pre_insn_back_chain = back_chain;
 
       /* A special case, first --- only recognized as the very first
@@ -1287,8 +1641,8 @@ s390_get_frame_info (CORE_ADDR start_pc,
          restore instructions.  (The back chain is never restored,
          just popped.)  */
       {
-        struct prologue_value *sp = &gpr[S390_SP_REGNUM - S390_GP0_REGNUM];
-        struct prologue_value *fp = &gpr[S390_FRAME_REGNUM - S390_GP0_REGNUM];
+        struct prologue_value *sp = &gpr[S390_SP_REGNUM - S390_R0_REGNUM];
+        struct prologue_value *fp = &gpr[S390_FRAME_REGNUM - S390_R0_REGNUM];
         
         if ((! pv_is_identical (&pre_insn_sp, sp)
              && ! pv_is_register (sp, S390_SP_REGNUM, 0))
@@ -1318,7 +1672,7 @@ s390_get_frame_info (CORE_ADDR start_pc,
        that strongly suggests that we're going to use that as our
        frame pointer register, not the SP.  */
     {
-      struct prologue_value *fp = &gpr[S390_FRAME_REGNUM - S390_GP0_REGNUM];
+      struct prologue_value *fp = &gpr[S390_FRAME_REGNUM - S390_R0_REGNUM];
 
       if (fp->kind == pv_register
           && fp->reg == S390_SP_REGNUM)
@@ -1349,7 +1703,7 @@ s390_get_frame_info (CORE_ADDR start_pc,
         else
           frame_base_regno = S390_SP_REGNUM;
 
-        frame_base = &gpr[frame_base_regno - S390_GP0_REGNUM];
+        frame_base = &gpr[frame_base_regno - S390_R0_REGNUM];
 
         /* We know the frame base address; if the value of whatever
            register it came from is a constant offset from the
@@ -1363,7 +1717,7 @@ s390_get_frame_info (CORE_ADDR start_pc,
     /* If the analysis said that the current SP value is the original
        value less some constant, then that constant is the frame size.  */
     {
-      struct prologue_value *sp = &gpr[S390_SP_REGNUM - S390_GP0_REGNUM];
+      struct prologue_value *sp = &gpr[S390_SP_REGNUM - S390_R0_REGNUM];
 
       if (sp->kind == pv_register
           && sp->reg == S390_SP_REGNUM)
@@ -1428,12 +1782,12 @@ s390_get_frame_info (CORE_ADDR start_pc,
        - the analysis gave us enough information to actually figure it
          out.  */
     if (fi
-        && get_frame_saved_regs (fi)
+        && deprecated_get_frame_saved_regs (fi)
         && original_sp != -1)
       {
         int slot_num;
         CORE_ADDR slot_addr;
-        CORE_ADDR *saved_regs = get_frame_saved_regs (fi);
+        CORE_ADDR *saved_regs = deprecated_get_frame_saved_regs (fi);
 
         /* Scan the spill array; if a spill slot says it holds the
            original value of some register, then record that slot's
@@ -1476,6 +1830,50 @@ s390_get_frame_info (CORE_ADDR start_pc,
   return result;
 }
 
+/* Return true if we are in the functin's epilogue, i.e. after the
+   instruction that destroyed the function's stack frame.  */
+static int
+s390_in_function_epilogue_p (struct gdbarch *gdbarch, CORE_ADDR pc)
+{
+  int word_size = gdbarch_ptr_bit (gdbarch) / 8;
+
+  /* In frameless functions, there's not frame to destroy and thus
+     we don't care about the epilogue.
+
+     In functions with frame, the epilogue sequence is a pair of
+     a LM-type instruction that restores (amongst others) the
+     return register %r14 and the stack pointer %r15, followed
+     by a branch 'br %r14' --or equivalent-- that effects the
+     actual return.
+
+     In that situation, this function needs to return 'true' in
+     exactly one case: when pc points to that branch instruction.
+
+     Thus we try to disassemble the one instructions immediately
+     preceeding pc and check whether it is an LM-type instruction
+     modifying the stack pointer.
+
+     Note that disassembling backwards is not reliable, so there
+     is a slight chance of false positives here ...  */
+
+  bfd_byte insn[6];
+  unsigned int r1, r3, b2;
+  int d2;
+
+  if (word_size == 4
+      && !read_memory_nobpt (pc - 4, insn, 4)
+      && is_rs (insn, op_lm, &r1, &r3, &d2, &b2)
+      && r3 == S390_SP_REGNUM - S390_R0_REGNUM)
+    return 1;
+
+  if (word_size == 8
+      && !read_memory_nobpt (pc - 6, insn, 6)
+      && is_rse (insn, op1_lmg, op2_lmg, &r1, &r3, &d2, &b2)
+      && r3 == S390_SP_REGNUM - S390_R0_REGNUM)
+    return 1;
+
+  return 0;
+}
 
 static int
 s390_check_function_end (CORE_ADDR pc)
@@ -1565,22 +1963,6 @@ s390_sniff_pc_function_start (CORE_ADDR pc, struct frame_info *fi)
 }
 
 
-
-static CORE_ADDR
-s390_function_start (struct frame_info *fi)
-{
-  CORE_ADDR function_start = 0;
-
-  if (get_frame_extra_info (fi) && get_frame_extra_info (fi)->initialised)
-    function_start = get_frame_extra_info (fi)->function_start;
-  else if (get_frame_pc (fi))
-    function_start = get_frame_func (fi);
-  return function_start;
-}
-
-
-
-
 static int
 s390_frameless_function_invocation (struct frame_info *fi)
 {
@@ -1662,8 +2044,8 @@ s390_is_sigreturn (CORE_ADDR pc, struct frame_info *sighandler_fi,
              *sigcaller_pc =
                ADDR_BITS_REMOVE ((CORE_ADDR)
                                  read_memory_integer (temp_sregs +
-                                                      DEPRECATED_REGISTER_BYTE (S390_PC_REGNUM),
-                                                      S390_PSW_ADDR_SIZE));
+                                                      DEPRECATED_REGISTER_BYTE (S390_PSWA_REGNUM),
+                                                      S390_GPR_SIZE));
            }
        }
       retval = 1;
@@ -1719,7 +2101,7 @@ s390_frame_init_saved_regs (struct frame_info *fi)
 
   int quick;
 
-  if (get_frame_saved_regs (fi) == NULL)
+  if (deprecated_get_frame_saved_regs (fi) == NULL)
     {
       /* zalloc memsets the saved regs */
       frame_saved_regs_zalloc (fi);
@@ -1754,10 +2136,10 @@ s390_frame_saved_pc_nofix (struct frame_info *fi)
     {
       get_frame_extra_info (fi)->saved_pc_valid = 1;
       if (get_frame_extra_info (fi)->good_prologue
-          && get_frame_saved_regs (fi)[S390_RETADDR_REGNUM])
+          && deprecated_get_frame_saved_regs (fi)[S390_RETADDR_REGNUM])
         get_frame_extra_info (fi)->saved_pc
           = ADDR_BITS_REMOVE (read_memory_integer
-                              (get_frame_saved_regs (fi)[S390_RETADDR_REGNUM],
+                              (deprecated_get_frame_saved_regs (fi)[S390_RETADDR_REGNUM],
                                S390_GPR_SIZE));
       else
         get_frame_extra_info (fi)->saved_pc
@@ -1832,7 +2214,7 @@ s390_frame_chain (struct frame_info *thisframe)
        {
          /* read sigregs,regs.gprs[11 or 15] */
          prev_fp = read_memory_integer (sregs +
-                                        DEPRECATED_REGISTER_BYTE (S390_GP0_REGNUM +
+                                        DEPRECATED_REGISTER_BYTE (S390_R0_REGNUM +
                                                        (prev_fextra_info.
                                                         frame_pointer_saved_pc
                                                         ? 11 : 15)),
@@ -1841,24 +2223,24 @@ s390_frame_chain (struct frame_info *thisframe)
        }
       else
        {
-         if (get_frame_saved_regs (thisframe))
+         if (deprecated_get_frame_saved_regs (thisframe))
            {
              int regno;
 
               if (prev_fextra_info.frame_pointer_saved_pc
-                  && get_frame_saved_regs (thisframe)[S390_FRAME_REGNUM])
+                  && deprecated_get_frame_saved_regs (thisframe)[S390_FRAME_REGNUM])
                 regno = S390_FRAME_REGNUM;
               else
                 regno = S390_SP_REGNUM;
 
-             if (get_frame_saved_regs (thisframe)[regno])
+             if (deprecated_get_frame_saved_regs (thisframe)[regno])
                 {
                   /* The SP's entry of `saved_regs' is special.  */
                   if (regno == S390_SP_REGNUM)
-                    prev_fp = get_frame_saved_regs (thisframe)[regno];
+                    prev_fp = deprecated_get_frame_saved_regs (thisframe)[regno];
                   else
                     prev_fp =
-                      read_memory_integer (get_frame_saved_regs (thisframe)[regno],
+                      read_memory_integer (deprecated_get_frame_saved_regs (thisframe)[regno],
                                            S390_GPR_SIZE);
                 }
            }
@@ -1872,95 +2254,6 @@ s390_frame_chain (struct frame_info *thisframe)
   out as our frames are similar to rs6000 there is a possibility
   i386 dosen't need it. */
 
-
-
-/* a given return value in `regbuf' with a type `valtype', extract and copy its
-   value into `valbuf' */
-static void
-s390_extract_return_value (struct type *valtype, char *regbuf, char *valbuf)
-{
-  /* floats and doubles are returned in fpr0. fpr's have a size of 8 bytes.
-     We need to truncate the return value into float size (4 byte) if
-     necessary. */
-  int len = TYPE_LENGTH (valtype);
-
-  if (TYPE_CODE (valtype) == TYPE_CODE_FLT)
-    memcpy (valbuf, &regbuf[DEPRECATED_REGISTER_BYTE (S390_FP0_REGNUM)], len);
-  else
-    {
-      int offset = 0;
-      /* return value is copied starting from r2. */
-      if (TYPE_LENGTH (valtype) < S390_GPR_SIZE)
-       offset = S390_GPR_SIZE - TYPE_LENGTH (valtype);
-      memcpy (valbuf,
-             regbuf + DEPRECATED_REGISTER_BYTE (S390_GP0_REGNUM + 2) + offset,
-             TYPE_LENGTH (valtype));
-    }
-}
-
-
-static char *
-s390_promote_integer_argument (struct type *valtype, char *valbuf,
-                              char *reg_buff, int *arglen)
-{
-  char *value = valbuf;
-  int len = TYPE_LENGTH (valtype);
-
-  if (len < S390_GPR_SIZE)
-    {
-      /* We need to upgrade this value to a register to pass it correctly */
-      int idx, diff = S390_GPR_SIZE - len, negative =
-       (!TYPE_UNSIGNED (valtype) && value[0] & 0x80);
-      for (idx = 0; idx < S390_GPR_SIZE; idx++)
-       {
-         reg_buff[idx] = (idx < diff ? (negative ? 0xff : 0x0) :
-                          value[idx - diff]);
-       }
-      value = reg_buff;
-      *arglen = S390_GPR_SIZE;
-    }
-  else
-    {
-      if (len & (S390_GPR_SIZE - 1))
-       {
-         fprintf_unfiltered (gdb_stderr,
-                             "s390_promote_integer_argument detected an argument not "
-                             "a multiple of S390_GPR_SIZE & greater than S390_GPR_SIZE "
-                             "we might not deal with this correctly.\n");
-       }
-      *arglen = len;
-    }
-
-  return (value);
-}
-
-static void
-s390_store_return_value (struct type *valtype, char *valbuf)
-{
-  int arglen;
-  char *reg_buff = alloca (max (S390_FPR_SIZE, DEPRECATED_REGISTER_SIZE)), *value;
-
-  if (TYPE_CODE (valtype) == TYPE_CODE_FLT)
-    {
-      if (TYPE_LENGTH (valtype) == 4
-          || TYPE_LENGTH (valtype) == 8)
-        deprecated_write_register_bytes (DEPRECATED_REGISTER_BYTE (S390_FP0_REGNUM),
-                                        valbuf, TYPE_LENGTH (valtype));
-      else
-        error ("GDB is unable to return `long double' values "
-               "on this architecture.");
-    }
-  else
-    {
-      value =
-       s390_promote_integer_argument (valtype, valbuf, reg_buff, &arglen);
-      /* Everything else is returned in GPR2 and up. */
-      deprecated_write_register_bytes (DEPRECATED_REGISTER_BYTE (S390_GP0_REGNUM + 2),
-                                      value, arglen);
-    }
-}
-
-
 /* Not the most efficent code in the world */
 static int
 s390_fp_regnum (void)
@@ -1992,22 +2285,22 @@ s390_pop_frame_regular (struct frame_info *frame)
   write_register (S390_PC_REGNUM, DEPRECATED_FRAME_SAVED_PC (frame));
 
   /* Restore any saved registers.  */
-  if (get_frame_saved_regs (frame))
+  if (deprecated_get_frame_saved_regs (frame))
     {
       for (regnum = 0; regnum < NUM_REGS; regnum++)
-        if (get_frame_saved_regs (frame)[regnum] != 0)
+        if (deprecated_get_frame_saved_regs (frame)[regnum] != 0)
           {
             ULONGEST value;
             
-            value = read_memory_unsigned_integer (get_frame_saved_regs (frame)[regnum],
-                                                  REGISTER_RAW_SIZE (regnum));
+            value = read_memory_unsigned_integer (deprecated_get_frame_saved_regs (frame)[regnum],
+                                                  DEPRECATED_REGISTER_RAW_SIZE (regnum));
             write_register (regnum, value);
           }
 
       /* Actually cut back the stack.  Remember that the SP's element of
          saved_regs is the old SP itself, not the address at which it is
          saved.  */
-      write_register (S390_SP_REGNUM, get_frame_saved_regs (frame)[S390_SP_REGNUM]);
+      write_register (S390_SP_REGNUM, deprecated_get_frame_saved_regs (frame)[S390_SP_REGNUM]);
     }
 
   /* Throw away any cached frame information.  */
@@ -2028,6 +2321,8 @@ s390_pop_frame (void)
 }
 
 
+/* Dummy function calls.  */
+
 /* Return non-zero if TYPE is an integer-like type, zero otherwise.
    "Integer-like" types are those that should be passed the way
    integers are: integers, enums, ranges, characters, and booleans.  */
@@ -2043,7 +2338,6 @@ is_integer_like (struct type *type)
           || code == TYPE_CODE_BOOL);
 }
 
-
 /* Return non-zero if TYPE is a pointer-like type, zero otherwise.
    "Pointer-like" types are those that should be passed the way
    pointers are: pointers and references.  */
@@ -2070,17 +2364,21 @@ is_pointer_like (struct type *type)
 
    ... and so on.
 
-   WHY THE HECK DO WE CARE ABOUT THIS???  Well, it turns out that GCC
-   passes all float singletons and double singletons as if they were
-   simply floats or doubles.  This is *not* what the ABI says it
-   should do.  */
+   All such structures are passed as if they were floats or doubles,
+   as the (revised) ABI says.  */
 static int
 is_float_singleton (struct type *type)
 {
-  return (TYPE_CODE (type) == TYPE_CODE_STRUCT
-          && TYPE_NFIELDS (type) == 1
-          && (TYPE_CODE (TYPE_FIELD_TYPE (type, 0)) == TYPE_CODE_FLT
-              || is_float_singleton (TYPE_FIELD_TYPE (type, 0))));
+  if (TYPE_CODE (type) == TYPE_CODE_STRUCT && TYPE_NFIELDS (type) == 1)
+    {
+      struct type *singleton_type = TYPE_FIELD_TYPE (type, 0);
+      CHECK_TYPEDEF (singleton_type);
+
+      return (TYPE_CODE (singleton_type) == TYPE_CODE_FLT
+             || is_float_singleton (singleton_type));
+    }
+
+  return 0;
 }
 
 
@@ -2119,74 +2417,51 @@ is_float_like (struct type *type)
 }
 
 
-/* Return non-zero if TYPE is considered a `DOUBLE_OR_FLOAT', as
-   defined by the parameter passing conventions described in the
-   "GNU/Linux for S/390 ELF Application Binary Interface Supplement".
-   Otherwise, return zero.  */
 static int
-is_double_or_float (struct type *type)
+is_power_of_two (unsigned int n)
 {
-  return (is_float_like (type)
-          && (TYPE_LENGTH (type) == 4
-              || TYPE_LENGTH (type) == 8));
+  return ((n & (n - 1)) == 0);
 }
 
-
-/* Return non-zero if TYPE is a `DOUBLE_ARG', as defined by the
-   parameter passing conventions described in the "GNU/Linux for S/390
-   ELF Application Binary Interface Supplement".  Return zero
-   otherwise.  */
+/* Return non-zero if TYPE should be passed as a pointer to a copy,
+   zero otherwise.  */
 static int
-is_double_arg (struct type *type)
+s390_function_arg_pass_by_reference (struct type *type)
 {
   unsigned length = TYPE_LENGTH (type);
+  if (length > 8)
+    return 1;
 
-  /* The s390x ABI doesn't handle DOUBLE_ARGS specially.  */
-  if (GDB_TARGET_IS_ESAME)
-    return 0;
-
-  return ((is_integer_like (type)
-           || is_struct_like (type))
-          && length == 8);
+  /* FIXME: All complex and vector types are also returned by reference.  */
+  return is_struct_like (type) && !is_power_of_two (length);
 }
 
-
-/* Return non-zero if TYPE is considered a `SIMPLE_ARG', as defined by
-   the parameter passing conventions described in the "GNU/Linux for
-   S/390 ELF Application Binary Interface Supplement".  Return zero
-   otherwise.  */
+/* Return non-zero if TYPE should be passed in a float register
+   if possible.  */
 static int
-is_simple_arg (struct type *type)
+s390_function_arg_float (struct type *type)
 {
   unsigned length = TYPE_LENGTH (type);
+  if (length > 8)
+    return 0;
 
-  /* This is almost a direct translation of the ABI's language, except
-     that we have to exclude 8-byte structs; those are DOUBLE_ARGs.  */
-  return ((is_integer_like (type) && length <= DEPRECATED_REGISTER_SIZE)
-          || is_pointer_like (type)
-          || (is_struct_like (type) && !is_double_arg (type)));
-}
-
-
-static int
-is_power_of_two (unsigned int n)
-{
-  return ((n & (n - 1)) == 0);
+  return is_float_like (type);
 }
 
-/* Return non-zero if TYPE should be passed as a pointer to a copy,
-   zero otherwise.  TYPE must be a SIMPLE_ARG, as recognized by
-   `is_simple_arg'.  */
+/* Return non-zero if TYPE should be passed in an integer register
+   (or a pair of integer registers) if possible.  */
 static int
-pass_by_copy_ref (struct type *type)
+s390_function_arg_integer (struct type *type)
 {
   unsigned length = TYPE_LENGTH (type);
+  if (length > 8)
+    return 0;
 
-  return (is_struct_like (type)
-          && !(is_power_of_two (length) && length <= DEPRECATED_REGISTER_SIZE));
+   return is_integer_like (type)
+         || is_pointer_like (type)
+         || (is_struct_like (type) && is_power_of_two (length));
 }
 
-
 /* Return ARG, a `SIMPLE_ARG', sign-extended or zero-extended to a full
    word as required for the ABI.  */
 static LONGEST
@@ -2206,28 +2481,6 @@ extend_simple_arg (struct value *arg)
 }
 
 
-/* Round ADDR up to the next N-byte boundary.  N must be a power of
-   two.  */
-static CORE_ADDR
-round_up (CORE_ADDR addr, int n)
-{
-  /* Check that N is really a power of two.  */
-  gdb_assert (n && (n & (n-1)) == 0);
-  return ((addr + n - 1) & -n);
-}
-
-
-/* Round ADDR down to the next N-byte boundary.  N must be a power of
-   two.  */
-static CORE_ADDR
-round_down (CORE_ADDR addr, int n)
-{
-  /* Check that N is really a power of two.  */
-  gdb_assert (n && (n & (n-1)) == 0);
-  return (addr & -n);
-}
-
-
 /* Return the alignment required by TYPE.  */
 static int
 alignment_of (struct type *type)
@@ -2279,66 +2532,45 @@ alignment_of (struct type *type)
    Our caller has taken care of any type promotions needed to satisfy
    prototypes or the old K&R argument-passing rules.  */
 static CORE_ADDR
-s390_push_arguments (int nargs, struct value **args, CORE_ADDR sp,
-                    int struct_return, CORE_ADDR struct_addr)
-{
+s390_push_dummy_call (struct gdbarch *gdbarch, CORE_ADDR func_addr,
+                     struct regcache *regcache, CORE_ADDR bp_addr,
+                     int nargs, struct value **args, CORE_ADDR sp,
+                     int struct_return, CORE_ADDR struct_addr)
+{
+  struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch);
+  int word_size = gdbarch_ptr_bit (gdbarch) / 8;
+  ULONGEST orig_sp;
   int i;
-  int pointer_size = (TARGET_PTR_BIT / TARGET_CHAR_BIT);
-
-  /* The number of arguments passed by reference-to-copy.  */
-  int num_copies;
 
   /* If the i'th argument is passed as a reference to a copy, then
      copy_addr[i] is the address of the copy we made.  */
   CORE_ADDR *copy_addr = alloca (nargs * sizeof (CORE_ADDR));
 
   /* Build the reference-to-copy area.  */
-  num_copies = 0;
   for (i = 0; i < nargs; i++)
     {
       struct value *arg = args[i];
       struct type *type = VALUE_TYPE (arg);
       unsigned length = TYPE_LENGTH (type);
 
-      if (is_simple_arg (type)
-          && pass_by_copy_ref (type))
+      if (s390_function_arg_pass_by_reference (type))
         {
           sp -= length;
-          sp = round_down (sp, alignment_of (type));
+          sp = align_down (sp, alignment_of (type));
           write_memory (sp, VALUE_CONTENTS (arg), length);
           copy_addr[i] = sp;
-          num_copies++;
         }
     }
 
   /* Reserve space for the parameter area.  As a conservative
      simplification, we assume that everything will be passed on the
-     stack.  */
-  {
-    int i;
-
-    for (i = 0; i < nargs; i++)
-      {
-        struct value *arg = args[i];
-        struct type *type = VALUE_TYPE (arg);
-        int length = TYPE_LENGTH (type);
-        
-        sp = round_down (sp, alignment_of (type));
+     stack.  Since every argument larger than 8 bytes will be 
+     passed by reference, we use this simple upper bound.  */
+  sp -= nargs * 8;
 
-        /* SIMPLE_ARG values get extended to DEPRECATED_REGISTER_SIZE bytes. 
-           Assume every argument is.  */
-        if (length < DEPRECATED_REGISTER_SIZE) length = DEPRECATED_REGISTER_SIZE;
-        sp -= length;
-      }
-  }
-
-  /* Include space for any reference-to-copy pointers.  */
-  sp = round_down (sp, pointer_size);
-  sp -= num_copies * pointer_size;
-    
   /* After all that, make sure it's still aligned on an eight-byte
      boundary.  */
-  sp = round_down (sp, 8);
+  sp = align_down (sp, 8);
 
   /* Finally, place the actual parameters, working from SP towards
      higher addresses.  The code above is supposed to reserve enough
@@ -2348,103 +2580,127 @@ s390_push_arguments (int nargs, struct value **args, CORE_ADDR sp,
     int gr = 2;
     CORE_ADDR starg = sp;
 
-    /* A struct is returned using general register 2 */
+    /* A struct is returned using general register 2 */
     if (struct_return)
-      gr++;
+      {
+       regcache_cooked_write_unsigned (regcache, S390_R0_REGNUM + gr,
+                                       struct_addr);
+       gr++;
+      }
 
     for (i = 0; i < nargs; i++)
       {
         struct value *arg = args[i];
         struct type *type = VALUE_TYPE (arg);
-        
-        if (is_double_or_float (type)
-            && fr <= S390_NUM_FP_PARAMETER_REGISTERS * 2 - 2)
-          {
-            /* When we store a single-precision value in an FP register,
-               it occupies the leftmost bits.  */
-            deprecated_write_register_bytes (DEPRECATED_REGISTER_BYTE (S390_FP0_REGNUM + fr),
-                                            VALUE_CONTENTS (arg),
-                                            TYPE_LENGTH (type));
-            fr += 2;
-          }
-        else if (is_simple_arg (type)
-                 && gr <= 6)
-          {
-            /* Do we need to pass a pointer to our copy of this
-               argument?  */
-            if (pass_by_copy_ref (type))
-              write_register (S390_GP0_REGNUM + gr, copy_addr[i]);
-            else
-              write_register (S390_GP0_REGNUM + gr, extend_simple_arg (arg));
-
-            gr++;
-          }
-        else if (is_double_arg (type)
-                 && gr <= 5)
-          {
-            deprecated_write_register_gen (S390_GP0_REGNUM + gr,
-                                          VALUE_CONTENTS (arg));
-            deprecated_write_register_gen (S390_GP0_REGNUM + gr + 1,
-                                          VALUE_CONTENTS (arg) + DEPRECATED_REGISTER_SIZE);
-            gr += 2;
-          }
-        else
-          {
-            /* The `OTHER' case.  */
-            enum type_code code = TYPE_CODE (type);
-            unsigned length = TYPE_LENGTH (type);
-            
-            /* If we skipped r6 because we couldn't fit a DOUBLE_ARG
-               in it, then don't go back and use it again later.  */
-            if (is_double_arg (type) && gr == 6)
-              gr = 7;
-
-            if (is_simple_arg (type))
-              {
-                /* Simple args are always extended to 
-                   DEPRECATED_REGISTER_SIZE bytes.  */
-                starg = round_up (starg, DEPRECATED_REGISTER_SIZE);
-
-                /* Do we need to pass a pointer to our copy of this
-                   argument?  */
-                if (pass_by_copy_ref (type))
-                  write_memory_signed_integer (starg, pointer_size,
-                                               copy_addr[i]);
-                else
-                  /* Simple args are always extended to 
-                     DEPRECATED_REGISTER_SIZE bytes. */
-                  write_memory_signed_integer (starg, DEPRECATED_REGISTER_SIZE,
-                                               extend_simple_arg (arg));
-                starg += DEPRECATED_REGISTER_SIZE;
-              }
-            else
-              {
-                /* You'd think we should say:
-                   starg = round_up (starg, alignment_of (type));
-                   Unfortunately, GCC seems to simply align the stack on
-                   a four/eight-byte boundary, even when passing doubles. */
-                starg = round_up (starg, S390_STACK_PARAMETER_ALIGNMENT);
-                write_memory (starg, VALUE_CONTENTS (arg), length);
-                starg += length;
-              }
-          }
+        unsigned length = TYPE_LENGTH (type);
+
+       if (s390_function_arg_pass_by_reference (type))
+         {
+           if (gr <= 6)
+             {
+               regcache_cooked_write_unsigned (regcache, S390_R0_REGNUM + gr,
+                                               copy_addr[i]);
+               gr++;
+             }
+           else
+             {
+               write_memory_unsigned_integer (starg, word_size, copy_addr[i]);
+               starg += word_size;
+             }
+         }
+       else if (s390_function_arg_float (type))
+         {
+           /* The GNU/Linux for S/390 ABI uses FPRs 0 and 2 to pass arguments,
+              the GNU/Linux for zSeries ABI uses 0, 2, 4, and 6.  */
+           if (fr <= (tdep->abi == ABI_LINUX_S390 ? 2 : 6))
+             {
+               /* When we store a single-precision value in an FP register,
+                  it occupies the leftmost bits.  */
+               regcache_cooked_write_part (regcache, S390_F0_REGNUM + fr,
+                                           0, length, VALUE_CONTENTS (arg));
+               fr += 2;
+             }
+           else
+             {
+               /* When we store a single-precision value in a stack slot,
+                  it occupies the rightmost bits.  */
+               starg = align_up (starg + length, word_size);
+                write_memory (starg - length, VALUE_CONTENTS (arg), length);
+             }
+         }
+       else if (s390_function_arg_integer (type) && length <= word_size)
+         {
+           if (gr <= 6)
+             {
+               /* Integer arguments are always extended to word size.  */
+               regcache_cooked_write_signed (regcache, S390_R0_REGNUM + gr,
+                                             extend_simple_arg (arg));
+               gr++;
+             }
+           else
+             {
+               /* Integer arguments are always extended to word size.  */
+               write_memory_signed_integer (starg, word_size,
+                                             extend_simple_arg (arg));
+                starg += word_size;
+             }
+         }
+       else if (s390_function_arg_integer (type) && length == 2*word_size)
+         {
+           if (gr <= 5)
+             {
+               regcache_cooked_write (regcache, S390_R0_REGNUM + gr,
+                                      VALUE_CONTENTS (arg));
+               regcache_cooked_write (regcache, S390_R0_REGNUM + gr + 1,
+                                      VALUE_CONTENTS (arg) + word_size);
+               gr += 2;
+             }
+           else
+             {
+               /* If we skipped r6 because we couldn't fit a DOUBLE_ARG
+                  in it, then don't go back and use it again later.  */
+               gr = 7;
+
+               write_memory (starg, VALUE_CONTENTS (arg), length);
+               starg += length;
+             }
+         }
+       else
+         internal_error (__FILE__, __LINE__, "unknown argument type");
       }
   }
 
   /* Allocate the standard frame areas: the register save area, the
      word reserved for the compiler (which seems kind of meaningless),
      and the back chain pointer.  */
-  sp -= S390_STACK_FRAME_OVERHEAD;
+  sp -= 16*word_size + 32;
 
   /* Write the back chain pointer into the first word of the stack
-     frame.  This will help us get backtraces from within functions
-     called from GDB.  */
-  write_memory_unsigned_integer (sp, (TARGET_PTR_BIT / TARGET_CHAR_BIT),
-                                 deprecated_read_fp ());
+     frame.  This is needed to unwind across a dummy frame.  */
+  regcache_cooked_read_unsigned (regcache, S390_SP_REGNUM, &orig_sp);
+  write_memory_unsigned_integer (sp, word_size, orig_sp);
+
+  /* Store return address.  */
+  regcache_cooked_write_unsigned (regcache, S390_RETADDR_REGNUM, bp_addr);
+  
+  /* Store updated stack pointer.  */
+  regcache_cooked_write_unsigned (regcache, S390_SP_REGNUM, sp);
 
+  /* Return stack pointer.  */ 
   return sp;
 }
 
+/* Assuming NEXT_FRAME->prev is a dummy, return the frame ID of that
+   dummy frame.  The frame ID's base needs to match the TOS value
+   returned by push_dummy_call, and the PC match the dummy frame's
+   breakpoint.  */
+static struct frame_id
+s390_unwind_dummy_id (struct gdbarch *gdbarch, struct frame_info *next_frame)
+{
+  ULONGEST sp;
+  frame_unwind_unsigned_register (next_frame, S390_SP_REGNUM, &sp);
+  return frame_id_build (sp, frame_pc_unwind (next_frame));
+}
 
 static CORE_ADDR
 s390_frame_align (struct gdbarch *gdbarch, CORE_ADDR addr)
@@ -2455,46 +2711,110 @@ s390_frame_align (struct gdbarch *gdbarch, CORE_ADDR addr)
 }
 
 
-static int
-s390_use_struct_convention (int gcc_p, struct type *value_type)
-{
-  enum type_code code = TYPE_CODE (value_type);
+/* Function return value access.  */
 
-  return (code == TYPE_CODE_STRUCT
-          || code == TYPE_CODE_UNION);
-}
+static enum return_value_convention
+s390_return_value_convention (struct gdbarch *gdbarch, struct type *type)
+{
+  int length = TYPE_LENGTH (type);
+  if (length > 8)
+    return RETURN_VALUE_STRUCT_CONVENTION;
 
+  switch (TYPE_CODE (type))
+    {
+    case TYPE_CODE_STRUCT:
+    case TYPE_CODE_UNION:
+    case TYPE_CODE_ARRAY:
+      return RETURN_VALUE_STRUCT_CONVENTION;
 
-/* Return the GDB type object for the "standard" data type
-   of data in register N.  */
-static struct type *
-s390_register_virtual_type (int regno)
-{
-  if (S390_FP0_REGNUM <= regno && regno < S390_FP0_REGNUM + S390_NUM_FPRS)
-    return builtin_type_double;
-  else
-    return builtin_type_int;
+    default:
+      return RETURN_VALUE_REGISTER_CONVENTION;
+    }
 }
 
-
-static struct type *
-s390x_register_virtual_type (int regno)
+static enum return_value_convention
+s390_return_value (struct gdbarch *gdbarch, struct type *type, 
+                  struct regcache *regcache, void *out, const void *in)
 {
-  return (regno == S390_FPC_REGNUM) ||
-    (regno >= S390_FIRST_ACR && regno <= S390_LAST_ACR) ? builtin_type_int :
-    (regno >= S390_FP0_REGNUM) ? builtin_type_double : builtin_type_long;
-}
+  int word_size = gdbarch_ptr_bit (gdbarch) / 8;
+  int length = TYPE_LENGTH (type);
+  enum return_value_convention rvc = 
+                       s390_return_value_convention (gdbarch, type);
+  if (in)
+    {
+      switch (rvc)
+       {
+       case RETURN_VALUE_REGISTER_CONVENTION:
+         if (TYPE_CODE (type) == TYPE_CODE_FLT)
+           {
+             /* When we store a single-precision value in an FP register,
+                it occupies the leftmost bits.  */
+             regcache_cooked_write_part (regcache, S390_F0_REGNUM, 
+                                         0, length, in);
+           }
+         else if (length <= word_size)
+           {
+             /* Integer arguments are always extended to word size.  */
+             if (TYPE_UNSIGNED (type))
+               regcache_cooked_write_unsigned (regcache, S390_R2_REGNUM,
+                       extract_unsigned_integer (in, length));
+             else
+               regcache_cooked_write_signed (regcache, S390_R2_REGNUM,
+                       extract_signed_integer (in, length));
+           }
+         else if (length == 2*word_size)
+           {
+             regcache_cooked_write (regcache, S390_R2_REGNUM, in);
+             regcache_cooked_write (regcache, S390_R3_REGNUM,
+                                    (const char *)in + word_size);
+           }
+         else
+           internal_error (__FILE__, __LINE__, "invalid return type");
+         break;
 
+       case RETURN_VALUE_STRUCT_CONVENTION:
+         error ("Cannot set function return value.");
+         break;
+       }
+    }
+  else if (out)
+    {
+      switch (rvc)
+       {
+       case RETURN_VALUE_REGISTER_CONVENTION:
+         if (TYPE_CODE (type) == TYPE_CODE_FLT)
+           {
+             /* When we store a single-precision value in an FP register,
+                it occupies the leftmost bits.  */
+             regcache_cooked_read_part (regcache, S390_F0_REGNUM, 
+                                        0, length, out);
+           }
+         else if (length <= word_size)
+           {
+             /* Integer arguments occupy the rightmost bits.  */
+             regcache_cooked_read_part (regcache, S390_R2_REGNUM, 
+                                        word_size - length, length, out);
+           }
+         else if (length == 2*word_size)
+           {
+             regcache_cooked_read (regcache, S390_R2_REGNUM, out);
+             regcache_cooked_read (regcache, S390_R3_REGNUM,
+                                   (char *)out + word_size);
+           }
+         else
+           internal_error (__FILE__, __LINE__, "invalid return type");
+         break;
 
+       case RETURN_VALUE_STRUCT_CONVENTION:
+         error ("Function return value unknown.");
+         break;
+       }
+    }
 
-static void
-s390_store_struct_return (CORE_ADDR addr, CORE_ADDR sp)
-{
-  write_register (S390_GP0_REGNUM + 2, addr);
+  return rvc;
 }
 
 
-
 static const unsigned char *
 s390_breakpoint_from_pc (CORE_ADDR *pcptr, int *lenptr)
 {
@@ -2531,14 +2851,6 @@ s390_addr_bits_remove (CORE_ADDR addr)
   return (addr) & 0x7fffffff;
 }
 
-
-static CORE_ADDR
-s390_push_return_address (CORE_ADDR pc, CORE_ADDR sp)
-{
-  write_register (S390_RETADDR_REGNUM, entry_point_address ());
-  return sp;
-}
-
 static int
 s390_address_class_type_flags (int byte_size, int dwarf2_addr_class)
 {
@@ -2573,10 +2885,8 @@ s390_address_class_name_to_type_flags (struct gdbarch *gdbarch, const char *name
 static struct gdbarch *
 s390_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
 {
-  static LONGEST s390_call_dummy_words[] = { 0 };
   struct gdbarch *gdbarch;
   struct gdbarch_tdep *tdep;
-  int elf_flags;
 
   /* First see if there is already a gdbarch that can satisfy the request.  */
   arches = gdbarch_list_lookup_by_info (arches, &info);
@@ -2588,21 +2898,18 @@ s390_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
     return NULL;               /* No; then it's not for us.  */
 
   /* Yes: create a new gdbarch for the specified machine type.  */
-  gdbarch = gdbarch_alloc (&info, NULL);
+  tdep = XCALLOC (1, struct gdbarch_tdep);
+  gdbarch = gdbarch_alloc (&info, tdep);
 
   /* 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);
+  set_gdbarch_deprecated_init_frame_pc (gdbarch, deprecated_init_frame_pc_default);
 
   set_gdbarch_believe_pcc_promotion (gdbarch, 0);
   set_gdbarch_char_signed (gdbarch, 0);
 
-  set_gdbarch_frame_args_skip (gdbarch, 0);
   set_gdbarch_deprecated_frame_chain (gdbarch, s390_frame_chain);
   set_gdbarch_deprecated_frame_init_saved_regs (gdbarch, s390_frame_init_saved_regs);
-  set_gdbarch_deprecated_store_struct_return (gdbarch, s390_store_struct_return);
-  set_gdbarch_deprecated_extract_return_value (gdbarch, s390_extract_return_value);
-  set_gdbarch_deprecated_store_return_value (gdbarch, s390_store_return_value);
   /* Amount PC must be decremented by after a breakpoint.  This is
      often the number of bytes returned by BREAKPOINT_FROM_PC but not
      always.  */
@@ -2610,74 +2917,73 @@ s390_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
   set_gdbarch_deprecated_pop_frame (gdbarch, s390_pop_frame);
   /* Stack grows downward.  */
   set_gdbarch_inner_than (gdbarch, core_addr_lessthan);
-  /* Offset from address of function to start of its code.
-     Zero on most machines.  */
-  set_gdbarch_function_start_offset (gdbarch, 0);
-  set_gdbarch_deprecated_max_register_raw_size (gdbarch, 8);
-  set_gdbarch_deprecated_max_register_virtual_size (gdbarch, 8);
   set_gdbarch_breakpoint_from_pc (gdbarch, s390_breakpoint_from_pc);
   set_gdbarch_skip_prologue (gdbarch, s390_skip_prologue);
   set_gdbarch_deprecated_init_extra_frame_info (gdbarch, s390_init_extra_frame_info);
   set_gdbarch_deprecated_init_frame_pc_first (gdbarch, s390_init_frame_pc_first);
   set_gdbarch_deprecated_target_read_fp (gdbarch, s390_read_fp);
+  set_gdbarch_in_function_epilogue_p (gdbarch, s390_in_function_epilogue_p);
   /* This function that tells us whether the function invocation represented
      by FI does not have a frame on the stack associated with it.  If it
      does not, FRAMELESS is set to 1, else 0.  */
-  set_gdbarch_frameless_function_invocation (gdbarch,
-                                            s390_frameless_function_invocation);
+  set_gdbarch_deprecated_frameless_function_invocation (gdbarch, s390_frameless_function_invocation);
   /* Return saved PC from a frame */
   set_gdbarch_deprecated_frame_saved_pc (gdbarch, s390_frame_saved_pc);
   /* DEPRECATED_FRAME_CHAIN takes a frame's nominal address and
      produces the frame's chain-pointer. */
   set_gdbarch_deprecated_frame_chain (gdbarch, s390_frame_chain);
   set_gdbarch_deprecated_saved_pc_after_call (gdbarch, s390_saved_pc_after_call);
-  set_gdbarch_deprecated_register_byte (gdbarch, s390_register_byte);
   set_gdbarch_pc_regnum (gdbarch, S390_PC_REGNUM);
   set_gdbarch_sp_regnum (gdbarch, S390_SP_REGNUM);
-  set_gdbarch_deprecated_fp_regnum (gdbarch, S390_FP_REGNUM);
-  set_gdbarch_fp0_regnum (gdbarch, S390_FP0_REGNUM);
+  set_gdbarch_deprecated_fp_regnum (gdbarch, S390_SP_REGNUM);
+  set_gdbarch_fp0_regnum (gdbarch, S390_F0_REGNUM);
   set_gdbarch_num_regs (gdbarch, S390_NUM_REGS);
-  set_gdbarch_cannot_fetch_register (gdbarch, s390_cannot_fetch_register);
-  set_gdbarch_cannot_store_register (gdbarch, s390_cannot_fetch_register);
-  set_gdbarch_use_struct_convention (gdbarch, s390_use_struct_convention);
+  set_gdbarch_num_pseudo_regs (gdbarch, S390_NUM_PSEUDO_REGS);
   set_gdbarch_register_name (gdbarch, s390_register_name);
-  set_gdbarch_stab_reg_to_regnum (gdbarch, s390_stab_reg_to_regnum);
-  set_gdbarch_dwarf_reg_to_regnum (gdbarch, s390_stab_reg_to_regnum);
-  set_gdbarch_dwarf2_reg_to_regnum (gdbarch, s390_stab_reg_to_regnum);
-  set_gdbarch_deprecated_extract_struct_value_address
-    (gdbarch, generic_cannot_extract_struct_value_address);
-
-  /* Parameters for inferior function calls.  */
-  set_gdbarch_deprecated_pc_in_call_dummy (gdbarch, deprecated_pc_in_call_dummy_at_entry_point);
+  set_gdbarch_register_type (gdbarch, s390_register_type);
+  set_gdbarch_stab_reg_to_regnum (gdbarch, s390_dwarf_reg_to_regnum);
+  set_gdbarch_dwarf_reg_to_regnum (gdbarch, s390_dwarf_reg_to_regnum);
+  set_gdbarch_dwarf2_reg_to_regnum (gdbarch, s390_dwarf_reg_to_regnum);
+  set_gdbarch_convert_register_p (gdbarch, s390_convert_register_p);
+  set_gdbarch_register_to_value (gdbarch, s390_register_to_value);
+  set_gdbarch_value_to_register (gdbarch, s390_value_to_register);
+  set_gdbarch_register_reggroup_p (gdbarch, s390_register_reggroup_p);
+  set_gdbarch_regset_from_core_section (gdbarch,
+                                        s390_regset_from_core_section);
+
+  /* Inferior function calls.  */
+  set_gdbarch_push_dummy_call (gdbarch, s390_push_dummy_call);
+  set_gdbarch_unwind_dummy_id (gdbarch, s390_unwind_dummy_id);
   set_gdbarch_frame_align (gdbarch, s390_frame_align);
-  set_gdbarch_deprecated_push_arguments (gdbarch, s390_push_arguments);
-  set_gdbarch_deprecated_save_dummy_frame_tos (gdbarch, generic_save_dummy_frame_tos);
-  set_gdbarch_deprecated_push_return_address (gdbarch,
-                                              s390_push_return_address);
-  set_gdbarch_deprecated_sizeof_call_dummy_words (gdbarch, sizeof (s390_call_dummy_words));
-  set_gdbarch_deprecated_call_dummy_words (gdbarch, s390_call_dummy_words);
+  set_gdbarch_return_value (gdbarch, s390_return_value);
 
   switch (info.bfd_arch_info->mach)
     {
     case bfd_mach_s390_31:
-      set_gdbarch_deprecated_register_size (gdbarch, 4);
-      set_gdbarch_deprecated_register_raw_size (gdbarch, s390_register_raw_size);
-      set_gdbarch_deprecated_register_virtual_size (gdbarch, s390_register_raw_size);
-      set_gdbarch_deprecated_register_virtual_type (gdbarch, s390_register_virtual_type);
+      tdep->abi = ABI_LINUX_S390;
+
+      tdep->gregset = &s390_gregset;
+      tdep->sizeof_gregset = s390_sizeof_gregset;
+      tdep->fpregset = &s390_fpregset;
+      tdep->sizeof_fpregset = s390_sizeof_fpregset;
 
       set_gdbarch_addr_bits_remove (gdbarch, s390_addr_bits_remove);
-      set_gdbarch_deprecated_register_bytes (gdbarch, S390_REGISTER_BYTES);
+      set_gdbarch_pseudo_register_read (gdbarch, s390_pseudo_register_read);
+      set_gdbarch_pseudo_register_write (gdbarch, s390_pseudo_register_write);
       break;
     case bfd_mach_s390_64:
-      set_gdbarch_deprecated_register_size (gdbarch, 8);
-      set_gdbarch_deprecated_register_raw_size (gdbarch, s390x_register_raw_size);
-      set_gdbarch_deprecated_register_virtual_size (gdbarch, s390x_register_raw_size);
-      set_gdbarch_deprecated_register_virtual_type (gdbarch, s390x_register_virtual_type);
+      tdep->abi = ABI_LINUX_ZSERIES;
+
+      tdep->gregset = &s390x_gregset;
+      tdep->sizeof_gregset = s390x_sizeof_gregset;
+      tdep->fpregset = &s390_fpregset;
+      tdep->sizeof_fpregset = s390_sizeof_fpregset;
 
       set_gdbarch_long_bit (gdbarch, 64);
       set_gdbarch_long_long_bit (gdbarch, 64);
       set_gdbarch_ptr_bit (gdbarch, 64);
-      set_gdbarch_deprecated_register_bytes (gdbarch, S390X_REGISTER_BYTES);
+      set_gdbarch_pseudo_register_read (gdbarch, s390x_pseudo_register_read);
+      set_gdbarch_pseudo_register_write (gdbarch, s390x_pseudo_register_write);
       set_gdbarch_address_class_type_flags (gdbarch,
                                             s390_address_class_type_flags);
       set_gdbarch_address_class_type_flags_to_name (gdbarch,
@@ -2687,9 +2993,6 @@ s390_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
       break;
     }
 
-  /* Should be using push_dummy_call.  */
-  set_gdbarch_deprecated_dummy_write_sp (gdbarch, deprecated_write_sp);
-
   set_gdbarch_print_insn (gdbarch, print_insn_s390);
 
   return gdbarch;
This page took 0.042816 seconds and 4 git commands to generate.