]> git.ipfire.org Git - thirdparty/binutils-gdb.git/blobdiff - gdb/mn10300-tdep.c
This commit was generated by cvs2svn to track changes on a CVS vendor
[thirdparty/binutils-gdb.git] / gdb / mn10300-tdep.c
index fe32fa08b2f98bc0366ccb044d678ad820940d18..6c43176193b80dee2174484f1677037608659953 100644 (file)
@@ -1,5 +1,5 @@
 /* Target-dependent code for the Matsushita MN10300 for GDB, the GNU debugger.
-   Copyright 1996, 1997 Free Software Foundation, Inc.
+   Copyright 1996, 1997, 1998 Free Software Foundation, Inc.
 
 This file is part of GDB.
 
@@ -28,278 +28,498 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include "gdbcore.h"
 #include "symfile.h"
 
-/* Info gleaned from scanning a function's prologue.  */
+static char *mn10300_generic_register_names[] = 
+{ "d0", "d1", "d2", "d3", "a0", "a1", "a2", "a3",
+  "sp", "pc", "mdr", "psw", "lir", "lar", "", "",
+  "", "", "", "", "", "", "", "",
+  "", "", "", "", "", "", "", "fp" };
 
-struct pifsr                   /* Info about one saved reg */
+char **mn10300_register_names = mn10300_generic_register_names;
+
+static CORE_ADDR mn10300_analyze_prologue PARAMS ((struct frame_info *fi,
+                                                 CORE_ADDR pc));
+
+/* Additional info used by the frame */
+
+struct frame_extra_info
 {
-  int framereg;                        /* Frame reg (SP or FP) */
-  int offset;                  /* Offset from framereg */
-  int reg;                     /* Saved register number */
+  int status;
+  int stack_size;
 };
 
-struct prologue_info
+static struct frame_info *analyze_dummy_frame PARAMS ((CORE_ADDR, CORE_ADDR));
+static struct frame_info *
+analyze_dummy_frame (pc, frame)
+     CORE_ADDR pc;
+     CORE_ADDR frame;
 {
-  int framereg;
-  int frameoffset;
-  int start_function;
-  struct pifsr *pifsrs;
-};
+  static struct frame_info *dummy = NULL;
+  if (dummy == NULL)
+    {
+      dummy = xmalloc (sizeof (struct frame_info));
+      dummy->saved_regs = xmalloc (SIZEOF_FRAME_SAVED_REGS);
+      dummy->extra_info = xmalloc (sizeof (struct frame_extra_info));
+    }
+  dummy->next = NULL;
+  dummy->prev = NULL;
+  dummy->pc = pc;
+  dummy->frame = frame;
+  dummy->extra_info->status = 0;
+  dummy->extra_info->stack_size = 0;
+  memset (dummy->saved_regs, '\000', SIZEOF_FRAME_SAVED_REGS);
+  mn10300_analyze_prologue (dummy, 0);
+  return dummy;
+}
 
-static CORE_ADDR mn10300_scan_prologue PARAMS ((CORE_ADDR pc, 
-                                            struct prologue_info *fs));
-\f
-/* Function: scan_prologue
-   Scan the prologue of the function that contains PC, and record what
-   we find in PI.  PI->fsr must be zeroed by the called.  Returns the
-   pc after the prologue.  Note that the addresses saved in pi->fsr
-   are actually just frame relative (negative offsets from the frame
-   pointer).  This is because we don't know the actual value of the
-   frame pointer yet.  In some circumstances, the frame pointer can't
-   be determined till after we have scanned the prologue.  */
+/* Values for frame_info.status */
 
-static CORE_ADDR
-mn10300_scan_prologue (pc, pi)
-     CORE_ADDR pc;
-     struct prologue_info *pi;
+#define MY_FRAME_IN_SP 0x1
+#define MY_FRAME_IN_FP 0x2
+#define NO_MORE_FRAMES 0x4
+
+
+/* Should call_function allocate stack space for a struct return?  */
+int
+mn10300_use_struct_convention (gcc_p, type)
+     int gcc_p;
+     struct type *type;
+{
+  return (TYPE_NFIELDS (type) > 1 || TYPE_LENGTH (type) > 8);
+}
+
+/* The breakpoint instruction must be the same size as the smallest
+   instruction in the instruction set.
+
+   The Matsushita mn10x00 processors have single byte instructions
+   so we need a single byte breakpoint.  Matsushita hasn't defined
+   one, so we defined it ourselves.  */
+
+unsigned char *
+mn10300_breakpoint_from_pc (bp_addr, bp_size)
+     CORE_ADDR *bp_addr;
+     int *bp_size;
 {
-  CORE_ADDR func_addr, prologue_end, current_pc;
-  struct pifsr *pifsr;
-  int fp_used;
+  static char breakpoint[] = {0xff};
+  *bp_size = 1;
+  return breakpoint;
+}
 
-  printf("mn10300_scan_prologue start\n");
 
-  /* First, figure out the bounds of the prologue so that we can limit the
-     search to something reasonable.  */
+/* Fix fi->frame if it's bogus at this point.  This is a helper
+   function for mn10300_analyze_prologue. */
 
-  if (find_pc_partial_function (pc, NULL, &func_addr, NULL))
+static void
+fix_frame_pointer (fi, stack_size)
+    struct frame_info *fi;
+    int stack_size;
+{
+  if (fi && fi->next == NULL)
     {
-      struct symtab_and_line sal;
+      if (fi->extra_info->status & MY_FRAME_IN_SP)
+       fi->frame = read_sp () - stack_size;
+      else if (fi->extra_info->status & MY_FRAME_IN_FP)
+       fi->frame = read_register (A3_REGNUM);
+    }
+}
 
-      sal = find_pc_line (func_addr, 0);
 
-      if (func_addr == entry_point_address ())
-       pi->start_function = 1;
-      else
-       pi->start_function = 0;
+/* Set offsets of registers saved by movm instruction.
+   This is a helper function for mn10300_analyze_prologue.  */
 
-#if 0
-      if (sal.line == 0)
-       prologue_end = pc;
-      else
-       prologue_end = sal.end;
-#else
-      prologue_end = pc;
-#endif
+static void
+set_movm_offsets (fi, movm_args)
+    struct frame_info *fi;
+    int movm_args;
+{
+  int offset = 0;
+
+  if (fi == NULL || movm_args == 0)
+    return;
+
+  if (movm_args & 0x10)
+    {
+      fi->saved_regs[A3_REGNUM] = fi->frame + offset;
+      offset += 4;
     }
-  else
-    {                          /* We're in the boondocks */
-      func_addr = pc - 100;
-      prologue_end = pc;
+  if (movm_args & 0x20)
+    {
+      fi->saved_regs[A2_REGNUM] = fi->frame + offset;
+      offset += 4;
+    }
+  if (movm_args & 0x40)
+    {
+      fi->saved_regs[D3_REGNUM] = fi->frame + offset;
+      offset += 4;
+    }
+  if (movm_args & 0x80)
+    {
+      fi->saved_regs[D2_REGNUM] = fi->frame + offset;
+      offset += 4;
     }
+}
 
-  prologue_end = min (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. */ 
+/* The main purpose of this file is dealing with prologues to extract
+   information about stack frames and saved registers.
 
-  pi->frameoffset = 0;
-  pi->framereg = SP_REGNUM;
-  fp_used = 0;
-  pifsr = pi->pifsrs;
+   For reference here's how prologues look on the mn10300:
 
-  for (current_pc = func_addr; current_pc < prologue_end; current_pc += 2)
-    {
-      int insn;
-
-      insn = read_memory_unsigned_integer (current_pc, 2);
-
-      if ((insn & 0x07c0) == 0x0780 /* jarl or jr */
-         || (insn & 0xffe0) == 0x0060 /* jmp */
-         || (insn & 0x0780) == 0x0580) /* branch */
-       break;                  /* Ran into end of prologue */
-      if ((insn & 0xffe0) == ((SP_REGNUM << 11) | 0x0240)) /* add <imm>,sp */
-       pi->frameoffset = ((insn & 0x1f) ^ 0x10) - 0x10;
-      else if (insn == ((SP_REGNUM << 11) | 0x0600 | SP_REGNUM)) /* addi <imm>,sp,sp */
-       pi->frameoffset = read_memory_integer (current_pc + 2, 2);
-      else if (insn == ((FP_REGNUM << 11) | 0x0000 | 12)) /* mov r12,fp */
-       {
-         fp_used = 1;
-         pi->framereg = FP_REGNUM;
-       }
-      else if ((insn & 0x07ff) == (0x0760 | SP_REGNUM) /* st.w <reg>,<offset>[sp] */
-              || (fp_used
-                  && (insn & 0x07ff) == (0x0760 | FP_REGNUM))) /* st.w <reg>,<offset>[fp] */
-       if (pifsr)
-         {
-           pifsr->framereg = insn & 0x1f;
-           pifsr->reg = (insn >> 11) & 0x1f; /* Extract <reg> */
+     With frame pointer:
+       movm [d2,d3,a2,a3],sp
+       mov sp,a3
+       add <size>,sp
 
-           pifsr->offset = read_memory_integer (current_pc + 2, 2) & ~1;
+     Without frame pointer:
+       movm [d2,d3,a2,a3],sp (if needed)
+        add <size>,sp
 
-           pifsr++;
-         }
+   One day we might keep the stack pointer constant, that won't
+   change the code for prologues, but it will make the frame
+   pointerless case much more common.  */
+       
+/* Analyze the prologue to determine where registers are saved,
+   the end of the prologue, etc etc.  Return the end of the prologue
+   scanned.
 
-      if ((insn & 0x0780) >= 0x0600) /* Four byte instruction? */
-       current_pc += 2;
-    }
+   We store into FI (if non-null) several tidbits of information:
 
-  if (pifsr)
-    pifsr->framereg = 0;       /* Tie off last entry */
+    * stack_size -- size of this stack frame.  Note that if we stop in
+    certain parts of the prologue/epilogue we may claim the size of the
+    current frame is zero.  This happens when the current frame has
+    not been allocated yet or has already been deallocated.
 
-  printf("mn10300_scan_prologue end \n");
+    * fsr -- Addresses of registers saved in the stack by this frame.
 
-  return current_pc;
-}
+    * status -- A (relatively) generic status indicator.  It's a bitmask
+    with the following bits: 
 
-/* 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().
+      MY_FRAME_IN_SP: The base of the current frame is actually in
+      the stack pointer.  This can happen for frame pointerless
+      functions, or cases where we're stopped in the prologue/epilogue
+      itself.  For these cases mn10300_analyze_prologue will need up
+      update fi->frame before returning or analyzing the register
+      save instructions.
 
-   Note that when we are called for the last frame (currently active frame),
-   that fi->pc 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 mn10300_frame_chain ()).
+      MY_FRAME_IN_FP: The base of the current frame is in the
+      frame pointer register ($a2).
 
-   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).  */
+      NO_MORE_FRAMES: Set this if the current frame is "start" or
+      if the first instruction looks like mov <imm>,sp.  This tells
+      frame chain to not bother trying to unwind past this frame.  */
 
-void
-mn10300_init_extra_frame_info (fi)
-     struct frame_info *fi;
+static CORE_ADDR
+mn10300_analyze_prologue (fi, pc)
+    struct frame_info *fi;
+    CORE_ADDR pc;
 {
-  struct prologue_info pi;
-  struct pifsr pifsrs[NUM_REGS + 1], *pifsr;
-  int reg;
+  CORE_ADDR func_addr, func_end, addr, stop;
+  CORE_ADDR stack_size;
+  int imm_size;
+  unsigned char buf[4];
+  int status, movm_args = 0;
+  char *name;
+
+  /* Use the PC in the frame if it's provided to look up the
+     start of this function.  */
+  pc = (fi ? fi->pc : pc);
+
+  /* Find the start of this function.  */
+  status = find_pc_partial_function (pc, &name, &func_addr, &func_end);
+
+  /* Do nothing if we couldn't find the start of this function or if we're
+     stopped at the first instruction in the prologue.  */
+  if (status == 0)
+    return pc;
+
+  /* If we're in start, then give up.  */
+  if (strcmp (name, "start") == 0)
+    {
+      if (fi != NULL)
+       fi->extra_info->status = NO_MORE_FRAMES;
+      return pc;
+    }
 
-  printf("mn10300_init_extra_frame_info start\n");
+  /* At the start of a function our frame is in the stack pointer.  */
+  if (fi)
+    fi->extra_info->status = MY_FRAME_IN_SP;
 
-  if (fi->next)
-    fi->pc = FRAME_SAVED_PC (fi->next);
+  /* Get the next two bytes into buf, we need two because rets is a two
+     byte insn and the first isn't enough to uniquely identify it.  */
+  status = read_memory_nobpt (pc, buf, 2);
+  if (status != 0)
+    return pc;
+
+  /* If we're physically on an "rets" instruction, then our frame has
+     already been deallocated.  Note this can also be true for retf
+     and ret if they specify a size of zero.
 
-  memset (fi->fsr.regs, '\000', sizeof fi->fsr.regs);
+     In this case fi->frame is bogus, we need to fix it.  */
+  if (fi && buf[0] == 0xf0 && buf[1] == 0xfc)
+    {
+      if (fi->next == NULL)
+       fi->frame = read_sp ();
+      return fi->pc;
+    }
 
-  /* The call dummy doesn't save any registers on the stack, so we can return
-     now.  */
-  if (PC_IN_CALL_DUMMY (fi->pc, fi->frame, fi->frame))
-      return;
+  /* Similarly if we're stopped on the first insn of a prologue as our
+     frame hasn't been allocated yet.  */
+  if (fi && fi->pc == func_addr)
+    {
+      if (fi->next == NULL)
+       fi->frame = read_sp ();
+      return fi->pc;
+    }
 
-  pi.pifsrs = pifsrs;
+  /* Figure out where to stop scanning.  */
+  stop = fi ? fi->pc : func_end;
 
-  mn10300_scan_prologue (fi->pc, &pi);
+  /* Don't walk off the end of the function.  */
+  stop = stop > func_end ? func_end : stop;
 
-  if (!fi->next && pi.framereg == SP_REGNUM)
-    fi->frame = read_register (pi.framereg) - pi.frameoffset;
+  /* Start scanning on the first instruction of this function.  */
+  addr = func_addr;
 
-  for (pifsr = pifsrs; pifsr->framereg; pifsr++)
+  /* Suck in two bytes.  */
+  status = read_memory_nobpt (addr, buf, 2);
+  if (status != 0)
     {
-      fi->fsr.regs[pifsr->reg] = pifsr->offset + fi->frame;
+      fix_frame_pointer (fi, 0);
+      return addr;
+    }
 
-      if (pifsr->framereg == SP_REGNUM)
-       fi->fsr.regs[pifsr->reg] += pi.frameoffset;
+  /* First see if this insn sets the stack pointer; if so, it's something
+     we won't understand, so quit now.   */
+  if (buf[0] == 0xf2 && (buf[1] & 0xf3) == 0xf0)
+    {
+      if (fi)
+       fi->extra_info->status = NO_MORE_FRAMES;
+      return addr;
     }
 
-  printf("mn10300_init_extra_frame_info end\n");
-}
+  /* Now look for movm [regs],sp, which saves the callee saved registers.
 
-/* 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 mn10300_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.  */
+     At this time we don't know if fi->frame is valid, so we only note
+     that we encountered a movm instruction.  Later, we'll set the entries
+     in fsr.regs as needed.  */
+  if (buf[0] == 0xcf)
+    {
+      /* Extract the register list for the movm instruction.  */
+      status = read_memory_nobpt (addr + 1, buf, 1);
+      movm_args = *buf;
 
-CORE_ADDR
-mn10300_frame_chain (fi)
-     struct frame_info *fi;
-{
-  struct prologue_info pi;
-  CORE_ADDR callers_pc, fp;
+      addr += 2;
+
+      /* Quit now if we're beyond the stop point.  */
+      if (addr >= stop)
+       {
+         /* Fix fi->frame since it's bogus at this point.  */
+         if (fi && fi->next == NULL)
+           fi->frame = read_sp ();
+
+         /* Note if/where callee saved registers were saved.  */
+         set_movm_offsets (fi, movm_args);
+         return addr;
+       }
+
+      /* Get the next two bytes so the prologue scan can continue.  */
+      status = read_memory_nobpt (addr, buf, 2);
+      if (status != 0)
+       {
+         /* Fix fi->frame since it's bogus at this point.  */
+         if (fi && fi->next == NULL)
+           fi->frame = read_sp ();
+
+         /* Note if/where callee saved registers were saved.  */
+         set_movm_offsets (fi, movm_args);
+         return addr;
+       }
+    }
+
+  /* Now see if we set up a frame pointer via "mov sp,a3" */
+  if (buf[0] == 0x3f)
+    {
+      addr += 1;
+
+      /* The frame pointer is now valid.  */
+      if (fi)
+       {
+         fi->extra_info->status |= MY_FRAME_IN_FP;
+         fi->extra_info->status &= ~MY_FRAME_IN_SP;
+       }
+
+      /* Quit now if we're beyond the stop point.  */
+      if (addr >= stop)
+       {
+         /* Fix fi->frame if it's bogus at this point.  */
+         fix_frame_pointer (fi, 0);
+
+         /* Note if/where callee saved registers were saved.  */
+         set_movm_offsets (fi, movm_args);
+         return addr;
+       }
+
+      /* Get two more bytes so scanning can continue.  */
+      status = read_memory_nobpt (addr, buf, 2);
+      if (status != 0)
+       {
+         /* Fix fi->frame if it's bogus at this point.  */
+         fix_frame_pointer (fi, 0);
+
+         /* Note if/where callee saved registers were saved.  */
+         set_movm_offsets (fi, movm_args);
+         return addr;
+       }
+    }
+  
+  /* Next we should allocate the local frame.  No more prologue insns
+     are found after allocating the local frame.
+       
+     Search for add imm8,sp (0xf8feXX)
+        or     add imm16,sp (0xfafeXXXX)
+        or     add imm32,sp (0xfcfeXXXXXXXX).
+       
+     If none of the above was found, then this prologue has no 
+     additional stack.  */
+
+  status = read_memory_nobpt (addr, buf, 2);
+  if (status != 0)
+    {
+      /* Fix fi->frame if it's bogus at this point.  */
+      fix_frame_pointer (fi, 0);
+
+      /* Note if/where callee saved registers were saved.  */
+      set_movm_offsets (fi, movm_args);
+      return addr;
+    }
 
-  printf("mn10300_frame_chain start\n"); 
+  imm_size = 0;
+  if (buf[0] == 0xf8 && buf[1] == 0xfe)
+    imm_size = 1;
+  else if (buf[0] == 0xfa && buf[1] == 0xfe)
+    imm_size = 2;
+  else if (buf[0] == 0xfc && buf[1] == 0xfe)
+    imm_size = 4;
+
+  if (imm_size != 0)
+    {
+      /* Suck in imm_size more bytes, they'll hold the size of the
+         current frame.  */
+      status = read_memory_nobpt (addr + 2, buf, imm_size);
+      if (status != 0)
+       {
+         /* Fix fi->frame if it's bogus at this point.  */
+         fix_frame_pointer (fi, 0);
 
-  /* 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 = mn10300_find_callers_reg (fi, FP_REGNUM);
-  if (PC_IN_CALL_DUMMY(callers_pc, fp, fp))
-    return fp; /* caller is call-dummy: return oldest value of FP */
+         /* Note if/where callee saved registers were saved.  */
+         set_movm_offsets (fi, movm_args);
+         return addr;
+       }
 
-  /* Caller is NOT a call-dummy, so everything else should just work.
-     Even if THIS frame is a call-dummy! */
-  pi.pifsrs = NULL;
+      /* Note the size of the stack in the frame info structure.  */
+      stack_size = extract_signed_integer (buf, imm_size);
+      if (fi)
+       fi->extra_info->stack_size = stack_size;
 
-  mn10300_scan_prologue (callers_pc, &pi);
+      /* We just consumed 2 + imm_size bytes.  */
+      addr += 2 + imm_size;
 
-  printf("mn10300_frame_chain end\n"); 
+      /* No more prologue insns follow, so begin preparation to return.  */
+      /* Fix fi->frame if it's bogus at this point.  */
+      fix_frame_pointer (fi, stack_size);
 
-  if (pi.start_function)
-    return 0;                  /* Don't chain beyond the start function */
+      /* Note if/where callee saved registers were saved.  */
+      set_movm_offsets (fi, movm_args);
+      return addr;
+    }
 
-  if (pi.framereg == FP_REGNUM)
-    return mn10300_find_callers_reg (fi, pi.framereg);
+  /* We never found an insn which allocates local stack space, regardless
+     this is the end of the prologue.  */
+  /* Fix fi->frame if it's bogus at this point.  */
+  fix_frame_pointer (fi, 0);
 
-  return fi->frame - pi.frameoffset;
+  /* Note if/where callee saved registers were saved.  */
+  set_movm_offsets (fi, movm_args);
+  return addr;
 }
+  
+/* Function: frame_chain
+   Figure out and return the caller's frame pointer given current
+   frame_info struct.
 
-/* 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.  */
+   We don't handle dummy frames yet but we would probably just return the
+   stack pointer that was in use at the time the function call was made?  */
 
 CORE_ADDR
-mn10300_find_callers_reg (fi, regnum)
+mn10300_frame_chain (fi)
      struct frame_info *fi;
-     int regnum;
 {
-  printf("mn10300_find_callers_reg\n"); 
+  struct frame_info *dummy;
+  /* Walk through the prologue to determine the stack size,
+     location of saved registers, end of the prologue, etc.  */
+  if (fi->extra_info->status == 0)
+    mn10300_analyze_prologue (fi, (CORE_ADDR)0);
 
-  for (; fi; fi = fi->next)
-    if (PC_IN_CALL_DUMMY (fi->pc, fi->frame, fi->frame))
-      return generic_read_register_dummy (fi->pc, fi->frame, regnum);
-    else if (fi->fsr.regs[regnum] != 0)
-      return read_memory_unsigned_integer (fi->fsr.regs[regnum], 
-                                          REGISTER_RAW_SIZE(regnum));
+  /* Quit now if mn10300_analyze_prologue set NO_MORE_FRAMES.  */
+  if (fi->extra_info->status & NO_MORE_FRAMES)
+    return 0;
 
-  return read_register (regnum);
-}
+  /* Now that we've analyzed our prologue, determine the frame
+     pointer for our caller.
 
-/* Function: skip_prologue
-   Return the address of the first code past the prologue of the function.  */
+       If our caller has a frame pointer, then we need to
+       find the entry value of $a3 to our function.
 
-CORE_ADDR
-mn10300_skip_prologue (pc)
-     CORE_ADDR pc;
-{
-  CORE_ADDR func_addr, func_end;
+        If fsr.regs[A3_REGNUM] is nonzero, then it's at the memory
+        location pointed to by fsr.regs[A3_REGNUM].
 
-  printf("mn10300_skip_prologue\n"); 
+        Else it's still in $a3.
 
-  /* See what the symbol table says */
+       If our caller does not have a frame pointer, then his
+       frame base is fi->frame + -caller's stack size.  */
+       
+  /* The easiest way to get that info is to analyze our caller's frame.
+     So we set up a dummy frame and call mn10300_analyze_prologue to
+     find stuff for us.  */
+  dummy = analyze_dummy_frame (FRAME_SAVED_PC (fi), fi->frame);
 
-  if (find_pc_partial_function (pc, NULL, &func_addr, &func_end))
+  if (dummy->extra_info->status & MY_FRAME_IN_FP)
+    {
+      /* Our caller has a frame pointer.  So find the frame in $a3 or
+         in the stack.  */
+      if (fi->saved_regs[A3_REGNUM])
+       return (read_memory_integer (fi->saved_regs[A3_REGNUM], REGISTER_SIZE));
+      else
+       return read_register (A3_REGNUM);
+    }
+  else
     {
-      struct symtab_and_line sal;
+      int adjust = 0;
 
-      sal = find_pc_line (func_addr, 0);
+      adjust += (fi->saved_regs[D2_REGNUM] ? 4 : 0);
+      adjust += (fi->saved_regs[D3_REGNUM] ? 4 : 0);
+      adjust += (fi->saved_regs[A2_REGNUM] ? 4 : 0);
+      adjust += (fi->saved_regs[A3_REGNUM] ? 4 : 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;
+      /* Our caller does not have a frame pointer.  So his frame starts
+        at the base of our frame (fi->frame) + register save space
+        + <his size>.  */
+      return fi->frame + adjust + -dummy->extra_info->stack_size;
     }
+}
+
+/* Function: skip_prologue
+   Return the address of the first inst past the prologue of the function.  */
 
-/* We can't find the start of this function, so there's nothing we can do. */
-  return pc;
+CORE_ADDR
+mn10300_skip_prologue (pc)
+     CORE_ADDR pc;
+{
+  /* We used to check the debug symbols, but that can lose if
+     we have a null prologue.  */
+  return mn10300_analyze_prologue (NULL, pc);
 }
 
+
 /* Function: pop_frame
    This routine gets called when either the user uses the `return'
    command, or the call dummy breakpoint gets hit.  */
@@ -310,32 +530,36 @@ mn10300_pop_frame (frame)
 {
   int regnum;
 
-  printf("mn10300_pop_frame start\n"); 
-
   if (PC_IN_CALL_DUMMY(frame->pc, frame->frame, frame->frame))
     generic_pop_dummy_frame ();
   else
     {
       write_register (PC_REGNUM, FRAME_SAVED_PC (frame));
 
+      /* Restore any saved registers.  */
       for (regnum = 0; regnum < NUM_REGS; regnum++)
-       if (frame->fsr.regs[regnum] != 0)
-         write_register (regnum,
-                         read_memory_unsigned_integer (frame->fsr.regs[regnum],
-                                                       REGISTER_RAW_SIZE(regnum)));
+       if (frame->saved_regs[regnum] != 0)
+         {
+           ULONGEST value;
+
+           value = read_memory_unsigned_integer (frame->saved_regs[regnum],
+                                                 REGISTER_RAW_SIZE (regnum));
+           write_register (regnum, value);
+         }
 
+      /* Actually cut back the stack.  */
       write_register (SP_REGNUM, FRAME_FP (frame));
+
+      /* Don't we need to set the PC?!?  XXX FIXME.  */
     }
 
+  /* Throw away any cached frame information.  */
   flush_cached_frames ();
-
-  printf("mn10300_pop_frame end\n"); 
 }
 
 /* Function: push_arguments
    Setup arguments for a call to the target.  Arguments go in
-   order on the stack.
-*/
+   order on the stack.  */
 
 CORE_ADDR
 mn10300_push_arguments (nargs, args, sp, struct_return, struct_addr)
@@ -347,29 +571,44 @@ mn10300_push_arguments (nargs, args, sp, struct_return, struct_addr)
 {
   int argnum = 0;
   int len = 0;
-  int stack_offset = 0;  /* copy args to this offset onto stack */
+  int stack_offset = 0;
+  int regsused = struct_return ? 1 : 0;
 
-  printf("mn10300_push_arguments start\n"); 
-
-  /* First, just for safety, make sure stack is aligned */
+  /* This should be a nop, but align the stack just in case something
+     went wrong.  Stacks are four byte aligned on the mn10300.  */
   sp &= ~3;
 
-  /* Now make space on the stack for the args. */
+  /* Now make space on the stack for the args.
+
+     XXX This doesn't appear to handle pass-by-invisible reference
+     arguments.  */
   for (argnum = 0; argnum < nargs; argnum++)
-    len += ((TYPE_LENGTH(VALUE_TYPE(args[argnum])) + 3) & ~3);
+    {
+      int arg_length = (TYPE_LENGTH (VALUE_TYPE (args[argnum])) + 3) & ~3;
+
+      while (regsused < 2 && arg_length > 0)
+       {
+         regsused++;
+         arg_length -= 4;
+       }
+      len += arg_length;
+    }
 
+  /* Allocate stack space.  */
   sp -= len;
 
+  regsused = struct_return ? 1 : 0;
   /* Push all arguments onto the stack. */
   for (argnum = 0; argnum < nargs; argnum++)
     {
       int len;
       char *val;
 
+      /* XXX Check this.  What about UNIONS?  */
       if (TYPE_CODE (VALUE_TYPE (*args)) == TYPE_CODE_STRUCT
          && TYPE_LENGTH (VALUE_TYPE (*args)) > 8)
        {
-         /* for now, pretend structs aren't special */
+         /* XXX Wrong, we want a pointer to this argument.  */
           len = TYPE_LENGTH (VALUE_TYPE (*args));
           val = (char *)VALUE_CONTENTS (*args);
        }
@@ -379,19 +618,27 @@ mn10300_push_arguments (nargs, args, sp, struct_return, struct_addr)
          val = (char *)VALUE_CONTENTS (*args);
        }
 
+      while (regsused < 2 && len > 0)
+       {
+         write_register (regsused, extract_unsigned_integer (val, 4));
+         val += 4;
+         len -= 4;
+         regsused++;
+       }
+
       while (len > 0)
        {
          write_memory (sp + stack_offset, val, 4);
-
          len -= 4;
          val += 4;
          stack_offset += 4;
        }
+
       args++;
     }
 
-  printf("mn10300_push_arguments end\n"); 
-
+  /* Make space for the flushback area.  */
+  sp -= 8;
   return sp;
 }
 
@@ -404,9 +651,24 @@ mn10300_push_return_address (pc, sp)
      CORE_ADDR pc;
      CORE_ADDR sp;
 {
-  printf("mn10300_push_return_address\n"); 
+  unsigned char buf[4];
 
-  /* write_register (RP_REGNUM, CALL_DUMMY_ADDRESS ()); */
+  store_unsigned_integer (buf, 4, CALL_DUMMY_ADDRESS ());
+  write_memory (sp - 4, buf, 4);
+  return sp - 4;
+}
+
+/* Function: store_struct_return (addr,sp)
+   Store the structure value return address for an inferior function
+   call.  */
+CORE_ADDR
+mn10300_store_struct_return (addr, sp)
+     CORE_ADDR addr;
+     CORE_ADDR sp;
+{
+  /* The structure return address is passed as the first argument.  */
+  write_register (0, addr);
   return sp;
 }
  
@@ -421,13 +683,14 @@ CORE_ADDR
 mn10300_frame_saved_pc (fi)
      struct frame_info *fi;
 {
-  printf("mn10300_frame_saved_pc\n"); 
+  int adjust = 0;
 
-/*  if (PC_IN_CALL_DUMMY(fi->pc, fi->frame, fi->frame)) */
-    return generic_read_register_dummy(fi->pc, fi->frame, PC_REGNUM);
-/*  else
-    return mn10300_find_callers_reg (fi, RP_REGNUM);
-*/
+  adjust += (fi->saved_regs[D2_REGNUM] ? 4 : 0);
+  adjust += (fi->saved_regs[D3_REGNUM] ? 4 : 0);
+  adjust += (fi->saved_regs[A2_REGNUM] ? 4 : 0);
+  adjust += (fi->saved_regs[A3_REGNUM] ? 4 : 0);
+
+  return (read_memory_integer (fi->frame + adjust, REGISTER_SIZE));
 }
 
 void
@@ -439,49 +702,90 @@ get_saved_register (raw_buffer, optimized, addrp, frame, regnum, lval)
      int regnum;
      enum lval_type *lval;
 {
-  printf("get_saved_register\n"); 
-
   generic_get_saved_register (raw_buffer, optimized, addrp, 
                              frame, regnum, lval);
 }
 
-/* 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
-   */
+/* Function: mn10300_init_extra_frame_info
+   Setup the frame's frame pointer, pc, and frame addresses for saved
+   registers.  Most of the work is done in mn10300_analyze_prologue().
+
+   Note that when we are called for the last frame (currently active frame),
+   that fi->pc and fi->frame will already be setup.  However, fi->frame will
+   be valid only if this routine uses FP.  For previous frames, fi-frame will
+   always be correct.  mn10300_analyze_prologue will fix fi->frame if
+   it's not valid.
 
-int
-mn10300_fix_call_dummy (dummy, sp, fun, nargs, args, type, gcc_p)
-     char *dummy;
-     CORE_ADDR sp;
-     CORE_ADDR fun;
-     int nargs;
-     value_ptr *args;
-     struct type *type;
-     int gcc_p;
+   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).  */
+
+void
+mn10300_init_extra_frame_info (fi)
+     struct frame_info *fi;
 {
-  long offset24;
+  if (fi->next)
+    fi->pc = FRAME_SAVED_PC (fi->next);
+
+  frame_saved_regs_zalloc (fi);
+  fi->extra_info = (struct frame_extra_info *)
+    frame_obstack_alloc (sizeof (struct frame_extra_info));
+
+  fi->extra_info->status = 0;
+  fi->extra_info->stack_size = 0;
 
-  printf("mn10300_fix_call_dummy start\n"); 
+  mn10300_analyze_prologue (fi, 0);
+}
 
-  offset24 = (long) fun - (long) entry_point_address ();
-  offset24 &= 0x3fffff;
-  offset24 |= 0xff800000;      /* jarl <offset24>, r31 */
+/* Function: mn10300_virtual_frame_pointer
+   Return the register that the function uses for a frame pointer, 
+   plus any necessary offset to be applied to the register before
+   any frame pointer offsets.  */
 
-  store_unsigned_integer ((unsigned int *)&dummy[2], 2, offset24 & 0xffff);
-  store_unsigned_integer ((unsigned int *)&dummy[0], 2, offset24 >> 16);
+void
+mn10300_virtual_frame_pointer (pc, reg, offset)
+     CORE_ADDR pc;
+     long *reg;
+     long *offset;
+{
+  struct frame_info *dummy = analyze_dummy_frame (pc, 0);
+  /* Set up a dummy frame_info, Analyze the prolog and fill in the
+     extra info.  */
+  /* Results will tell us which type of frame it uses.  */
+  if (dummy->extra_info->status & MY_FRAME_IN_SP)
+    {
+      *reg    = SP_REGNUM;
+      *offset = -(dummy->extra_info->stack_size);
+    }
+  else
+    {
+      *reg    = A3_REGNUM;
+      *offset = 0;
+    }
+}
+  
+/* This can be made more generic later.  */
+static void
+set_machine_hook (filename)
+     char *filename;
+{
+  int i;
 
-  printf("mn10300_fix_call_dummy end\n"); 
+  if (bfd_get_mach (exec_bfd) == bfd_mach_mn10300
+      || bfd_get_mach (exec_bfd) == 0)
+    {
+      mn10300_register_names = mn10300_generic_register_names;
+    }
 
-  return 0;
 }
 
 void
 _initialize_mn10300_tdep ()
 {
-  printf("_initialize_mn10300_tdep\n"); 
+/*  printf("_initialize_mn10300_tdep\n"); */
 
   tm_print_insn = print_insn_mn10300;
+
+  specify_exec_file_hook (set_machine_hook);
 }
+