* remote-mips.c (mips_request): Use unsigned long during parsing
[deliverable/binutils-gdb.git] / gdb / mips-tdep.c
index 75f10d1e5612f4aef68cc623c02a1d3c70af52c8..57f3bd76aa486a69f5129876e6543a85d57e2726 100644 (file)
@@ -41,12 +41,6 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 /* FIXME: Put this declaration in frame.h.  */
 extern struct obstack frame_cache_obstack;
 
-/* FIXME! this code assumes 4-byte instructions.  */
-#define MIPS_INSTLEN 4         /* Length of an instruction */
-#define MIPS16_INSTLEN 2       /* Length of an instruction on MIPS16*/
-#define MIPS_NUMREGS 32                /* Number of integer or float registers */
-typedef unsigned long t_inst;  /* Integer big enough to hold an instruction */
-
 #if 0
 static int mips_in_lenient_prologue PARAMS ((CORE_ADDR, CORE_ADDR));
 #endif
@@ -196,6 +190,7 @@ static unsigned int heuristic_fence_post = 0;
 #define PROC_HIGH_ADDR(proc) ((proc)->high_addr) /* upper address bound */
 #define PROC_FRAME_OFFSET(proc) ((proc)->pdr.frameoffset)
 #define PROC_FRAME_REG(proc) ((proc)->pdr.framereg)
+#define PROC_FRAME_ADJUST(proc)  ((proc)->frame_adjust)
 #define PROC_REG_MASK(proc) ((proc)->pdr.regmask)
 #define PROC_FREG_MASK(proc) ((proc)->pdr.fregmask)
 #define PROC_REG_OFFSET(proc) ((proc)->pdr.regoffset)
@@ -213,6 +208,28 @@ struct linked_proc_info
 } *linked_proc_desc_table = NULL;
 
 
+/* Tell if the program counter value in MEMADDR is in a MIPS16 function.  */
+
+static int
+pc_is_mips16 (bfd_vma memaddr)
+{
+  struct minimal_symbol *sym;
+
+  /* If bit 0 of the address is set, assume this is a MIPS16 address. */
+  if (IS_MIPS16_ADDR (memaddr))
+    return 1;
+
+  /* A flag indicating that this is a MIPS16 function is stored by elfread.c in
+     the high bit of the info field.  Use this to decide if the function is
+     MIPS16 or normal MIPS.  */
+  sym = lookup_minimal_symbol_by_pc (memaddr);
+  if (sym)
+    return MSYMBOL_IS_SPECIAL (sym);
+  else
+    return 0;
+}
+
+
 /* This returns the PC of the first inst after the prologue.  If we can't
    find the prologue, then return 0.  */
 
@@ -307,9 +324,35 @@ mips16_decode_reg_save (inst, gen_mask)
     }
   else if ((inst & 0xff00) == 0x6200           /* sw $ra,n($sp) */
           || (inst & 0xff00) == 0xfa00)        /* sd $ra,n($sp) */
-    *gen_mask |= (1 << 31);
+    *gen_mask |= (1 << RA_REGNUM);
 }
 
+
+/* Fetch and return instruction from the specified location.  If the PC
+   is odd, assume it's a MIPS16 instruction; otherwise MIPS32.  */
+
+static t_inst
+mips_fetch_instruction (addr)
+    CORE_ADDR addr;
+{
+  char buf[MIPS_INSTLEN];
+  int instlen;
+  int status;
+
+  if (pc_is_mips16 (addr))
+    {
+      instlen = MIPS16_INSTLEN;
+      addr = UNMAKE_MIPS16_ADDR (addr);
+    }
+  else
+      instlen = MIPS_INSTLEN;
+  status = read_memory_nobpt (addr, buf, instlen);
+  if (status)
+    memory_error (status, addr);
+  return extract_unsigned_integer (buf, instlen);
+}
+
+
 /* Guaranteed to set fci->saved_regs to some values (it never leaves it
    NULL).  */
 
@@ -324,6 +367,7 @@ mips_find_saved_regs (fci)
   /* What registers have been saved?  Bitmasks.  */
   unsigned long gen_mask, float_mask;
   mips_extra_func_info_t proc_desc;
+  t_inst inst;
 
   fci->saved_regs = (struct frame_saved_regs *)
     obstack_alloc (&frame_cache_obstack, sizeof(struct frame_saved_regs));
@@ -400,32 +444,22 @@ mips_find_saved_regs (fci)
         claims are saved have been saved yet.  */
 
       CORE_ADDR addr;
-      int status;
-      char buf[MIPS_INSTLEN];
-      t_inst inst;
-      int instlen;
 
       /* Bitmasks; set if we have found a save for the register.  */
       unsigned long gen_save_found = 0;
       unsigned long float_save_found = 0;
+      int instlen;
 
-      if ((addr = PROC_LOW_ADDR (proc_desc)) & 1)
-       {
-         instlen = MIPS16_INSTLEN;     /* MIPS16 */
-         addr &= ~1;
-       }
-      else
-       instlen = MIPS_INSTLEN;         /* MIPS32 */
+      /* If the address is odd, assume this is MIPS16 code.  */
+      addr = PROC_LOW_ADDR (proc_desc);
+      instlen = pc_is_mips16 (addr) ? MIPS16_INSTLEN : MIPS_INSTLEN;
 
       /* Scan through this function's instructions preceding the current
          PC, and look for those that save registers.  */
       while (addr < fci->pc)
        {
-         status = read_memory_nobpt (addr, buf, instlen);
-         if (status)
-           memory_error (status, addr);
-         inst = extract_unsigned_integer (buf, instlen);
-         if (instlen == MIPS16_INSTLEN)
+         inst = mips_fetch_instruction (addr);
+         if (pc_is_mips16 (addr))
            mips16_decode_reg_save (inst, &gen_save_found);
          else
            mips32_decode_reg_save (inst, &gen_save_found, &float_save_found);
@@ -444,6 +478,33 @@ mips_find_saved_regs (fci)
        fci->saved_regs->regs[ireg] = reg_position;
        reg_position -= MIPS_REGSIZE;
       }
+
+  /* The MIPS16 entry instruction saves $s0 and $s1 in the reverse order
+     of that normally used by gcc.  Therefore, we have to fetch the first
+     instruction of the function, and if it's an entry instruction that
+     saves $s0 or $s1, correct their saved addresses.  */
+  if (pc_is_mips16 (PROC_LOW_ADDR (proc_desc)))
+    {
+      inst = mips_fetch_instruction (PROC_LOW_ADDR (proc_desc));
+      if ((inst & 0xf81f) == 0xe809 && (inst & 0x700) != 0x700) /* entry */
+       {
+         int reg;
+         int sreg_count = (inst >> 6) & 3;
+         
+         /* Check if the ra register was pushed on the stack.  */
+         reg_position = fci->frame + PROC_REG_OFFSET (proc_desc);
+         if (inst & 0x20)
+           reg_position -= MIPS_REGSIZE;
+
+         /* Check if the s0 and s1 registers were pushed on the stack.  */
+         for (reg = 16; reg < sreg_count+16; reg++)
+           {
+             fci->saved_regs->regs[reg] = reg_position;
+             reg_position -= MIPS_REGSIZE;
+           }
+       }
+    }
+
   /* Fill in the offsets for the registers which float_mask says
      were saved.  */
   reg_position = fci->frame + PROC_FREG_OFFSET (proc_desc);
@@ -451,7 +512,7 @@ mips_find_saved_regs (fci)
   /* The freg_offset points to where the first *double* register
      is saved.  So skip to the high-order word. */
   if (! GDB_TARGET_IS_MIPS64)
-    reg_position += 4;
+    reg_position += MIPS_REGSIZE;
 
   /* Fill in the offsets for the float registers which float_mask says
      were saved.  */
@@ -495,9 +556,9 @@ mips_addr_bits_remove (addr)
 {
 #if GDB_TARGET_IS_MIPS64
   if ((addr >> 32 == (CORE_ADDR)0xffffffff)
-      && (strcmp(target_shortname,"pmon")==0
-        || strcmp(target_shortname,"ddb")==0
-        || strcmp(target_shortname,"sim")==0))
+      && (strcmp (target_shortname,"pmon")==0
+        || strcmp (target_shortname,"ddb")==0
+        || strcmp (target_shortname,"sim")==0))
     {
       /* This hack is a work-around for existing boards using PMON,
         the simulator, and any other 64-bit targets that doesn't have
@@ -524,6 +585,20 @@ mips_addr_bits_remove (addr)
   return addr;
 }
 
+void
+mips_init_frame_pc_first (fromleaf, prev)
+     int fromleaf;
+     struct frame_info *prev;
+{
+  CORE_ADDR pc, tmp;
+
+  pc = ((fromleaf) ? SAVED_PC_AFTER_CALL (prev->next) :
+        prev->next ? FRAME_SAVED_PC (prev->next) : read_pc ());
+  tmp = mips_skip_stub (pc);
+  prev->pc =  tmp ? tmp : pc;
+}
+
+
 CORE_ADDR
 mips_frame_saved_pc(frame)
      struct frame_info *frame;
@@ -546,6 +621,21 @@ mips_frame_saved_pc(frame)
 static struct mips_extra_func_info temp_proc_desc;
 static struct frame_saved_regs temp_saved_regs;
 
+/* Set a register's saved stack address in temp_saved_regs.  If an address
+   has already been set for this register, do nothing; this way we will
+   only recognize the first save of a given register in a function prologue.
+   This is a helper function for mips{16,32}_heuristic_proc_desc.  */
+
+static void
+set_reg_offset (regno, offset)
+     int regno;
+     CORE_ADDR offset;
+{
+  if (temp_saved_regs.regs[regno] == 0)
+    temp_saved_regs.regs[regno] = offset;
+}
+
+
 /* This fencepost looks highly suspicious to me.  Removing it also
    seems suspicious as it could affect remote debugging across serial
    lines.  */
@@ -554,18 +644,21 @@ static CORE_ADDR
 heuristic_proc_start(pc)
     CORE_ADDR pc;
 {
-    CORE_ADDR start_pc = pc;
-    CORE_ADDR fence = start_pc - heuristic_fence_post;
+    CORE_ADDR start_pc;
+    CORE_ADDR fence;
     int instlen;
     int seen_adjsp = 0;
 
+    pc = ADDR_BITS_REMOVE (pc);
+    start_pc = pc;
+    fence = start_pc - heuristic_fence_post;
     if (start_pc == 0) return 0;
 
     if (heuristic_fence_post == UINT_MAX
        || fence < VM_MIN_ADDRESS)
       fence = VM_MIN_ADDRESS;
 
-    instlen = pc & 1 ? MIPS16_INSTLEN : MIPS_INSTLEN;
+    instlen = pc_is_mips16 (pc) ? MIPS16_INSTLEN : MIPS_INSTLEN;
 
     /* search back for previous return */
     for (start_pc -= instlen; ; start_pc -= instlen)
@@ -584,7 +677,7 @@ heuristic_proc_start(pc)
                else
                  warning("Hit heuristic-fence-post without finding");
                
-               warning("enclosing function for address 0x%s", paddr (pc));
+               warning("enclosing function for address 0x%s", paddr_nz (pc));
                if (!blurb_printed)
                  {
                    printf_filtered ("\
@@ -600,15 +693,17 @@ Otherwise, you told GDB there was a function where there isn't one, or\n\
 
            return 0; 
          }
-       else if (start_pc & 1)
+       else if (pc_is_mips16 (start_pc))
          {
+           unsigned short inst;
+
            /* On MIPS16, any one of the following is likely to be the
               start of a function:
                 entry
                 addiu sp,-n
                 daddiu sp,-n
                 extend -n followed by 'addiu sp,+n' or 'daddiu sp,+n'  */
-           unsigned short inst = read_memory_integer (start_pc & ~1, 2);
+           inst = mips_fetch_instruction (start_pc);
            if (((inst & 0xf81f) == 0xe809 && (inst & 0x700) != 0x700) /* entry */
                || (inst & 0xff80) == 0x6380    /* addiu sp,-n */
                || (inst & 0xff80) == 0xfb80    /* daddiu sp,-n */
@@ -634,7 +729,7 @@ Otherwise, you told GDB there was a function where there isn't one, or\n\
     return start_pc;
 }
 
-/* Fetch the immediate value from the current instruction.
+/* Fetch the immediate value from a MIPS16 instruction.
    If the previous instruction was an EXTEND, use it to extend
    the upper bits of the immediate value.  This is a helper function
    for mips16_heuristic_proc_desc.  */
@@ -642,7 +737,7 @@ Otherwise, you told GDB there was a function where there isn't one, or\n\
 static int
 mips16_get_imm (prev_inst, inst, nbits, scale, is_signed)
     unsigned short prev_inst;  /* previous instruction */
-    unsigned short inst;       /* current current instruction */
+    unsigned short inst;       /* current instruction */
     int nbits;                 /* number of bits in imm field */
     int scale;                 /* scale factor to be applied to imm */
     int is_signed;             /* is the imm field signed? */
@@ -651,7 +746,7 @@ mips16_get_imm (prev_inst, inst, nbits, scale, is_signed)
 
   if ((prev_inst & 0xf800) == 0xf000)  /* prev instruction was EXTEND? */
     {
-      offset = ((prev_inst & 0x1f) << 11) | (prev_inst & 0x7e00);
+      offset = ((prev_inst & 0x1f) << 11) | (prev_inst & 0x7e0);
       if (offset & 0x8000)             /* check for negative extend */
        offset = 0 - (0x10000 - (offset & 0xffff));
       return offset | (inst & 0x1f);
@@ -683,23 +778,20 @@ mips16_heuristic_proc_desc(start_pc, limit_pc, next_frame, sp)
   CORE_ADDR frame_addr = 0;    /* Value of $r17, used as frame pointer */
   unsigned short prev_inst = 0;        /* saved copy of previous instruction */
   unsigned inst = 0;           /* current instruction */
+  unsigned entry_inst = 0;     /* the entry instruction */
+  int reg, offset;
 
-  PROC_FRAME_OFFSET(&temp_proc_desc) = 0;
+  PROC_FRAME_OFFSET(&temp_proc_desc) = 0;      /* size of stack frame */
+  PROC_FRAME_ADJUST(&temp_proc_desc) = 0;      /* offset of FP from SP */
 
   for (cur_pc = start_pc; cur_pc < limit_pc; cur_pc += MIPS16_INSTLEN)
     {
-      char buf[MIPS16_INSTLEN];
-      int status, reg, offset;
-
       /* Save the previous instruction.  If it's an EXTEND, we'll extract
          the immediate offset extension from it in mips16_get_imm.  */
       prev_inst = inst;
 
-      /* Fetch the instruction.   */
-      status = read_memory_nobpt (cur_pc & ~1, buf, MIPS16_INSTLEN);
-      if (status) memory_error (status, cur_pc);
-      inst = (unsigned short) extract_unsigned_integer (buf, MIPS16_INSTLEN);
-
+      /* Fetch and decode the instruction.   */
+      inst = (unsigned short) mips_fetch_instruction (cur_pc);
       if ((inst & 0xff00) == 0x6300            /* addiu sp */
          || (inst & 0xff00) == 0xfb00)         /* daddiu sp */
        {
@@ -717,78 +809,102 @@ mips16_heuristic_proc_desc(start_pc, limit_pc, next_frame, sp)
          offset = mips16_get_imm (prev_inst, inst, 8, 4, 0);
          reg = mips16_to_32_reg[(inst & 0x700) >> 8];
          PROC_REG_MASK(&temp_proc_desc) |= (1 << reg);
-         temp_saved_regs.regs[reg] = sp + offset;
+         set_reg_offset (reg, sp + offset);
        }
       else if ((inst & 0xff00) == 0xf900)      /* sd reg,n($sp) */
        {
          offset = mips16_get_imm (prev_inst, inst, 5, 8, 0);
          reg = mips16_to_32_reg[(inst & 0xe0) >> 5];
          PROC_REG_MASK(&temp_proc_desc) |= (1 << reg);
-         temp_saved_regs.regs[reg] = sp + offset;
+         set_reg_offset (reg, sp + offset);
        }
       else if ((inst & 0xff00) == 0x6200)      /* sw $ra,n($sp) */
        {
          offset = mips16_get_imm (prev_inst, inst, 8, 4, 0);
-         PROC_REG_MASK(&temp_proc_desc) |= (1 << 31);
-         temp_saved_regs.regs[31] = sp + offset;
+         PROC_REG_MASK(&temp_proc_desc) |= (1 << RA_REGNUM);
+         set_reg_offset (RA_REGNUM, sp + offset);
        }
       else if ((inst & 0xff00) == 0xfa00)      /* sd $ra,n($sp) */
        {
          offset = mips16_get_imm (prev_inst, inst, 8, 8, 0);
-         PROC_REG_MASK(&temp_proc_desc) |= (1 << 31);
-         temp_saved_regs.regs[31] = sp + offset;
+         PROC_REG_MASK(&temp_proc_desc) |= (1 << RA_REGNUM);
+         set_reg_offset (RA_REGNUM, sp + offset);
        }
       else if (inst == 0x673d)                 /* move $s1, $sp */
        {
-         frame_addr = read_next_frame_reg(next_frame, 30);
+         frame_addr = sp;
          PROC_FRAME_REG (&temp_proc_desc) = 17;
        }
+      else if ((inst & 0xff00) == 0x0100)      /* addiu $s1,sp,n */
+       {
+         offset = mips16_get_imm (prev_inst, inst, 8, 4, 0);
+         frame_addr = sp + offset;
+         PROC_FRAME_REG (&temp_proc_desc) = 17;
+         PROC_FRAME_ADJUST (&temp_proc_desc) = offset;
+       }
       else if ((inst & 0xFF00) == 0xd900)      /* sw reg,offset($s1) */
        {
          offset = mips16_get_imm (prev_inst, inst, 5, 4, 0);
          reg = mips16_to_32_reg[(inst & 0xe0) >> 5];
          PROC_REG_MASK(&temp_proc_desc) |= 1 << reg;
-         temp_saved_regs.regs[reg] = frame_addr + offset;
+         set_reg_offset (reg, frame_addr + offset);
        }
       else if ((inst & 0xFF00) == 0x7900)      /* sd reg,offset($s1) */
        {
          offset = mips16_get_imm (prev_inst, inst, 5, 8, 0);
          reg = mips16_to_32_reg[(inst & 0xe0) >> 5];
          PROC_REG_MASK(&temp_proc_desc) |= 1 << reg;
-         temp_saved_regs.regs[reg] = frame_addr + offset;
+         set_reg_offset (reg, frame_addr + offset);
        }
       else if ((inst & 0xf81f) == 0xe809 && (inst & 0x700) != 0x700) /* entry */
-       {
-         int areg_count = (inst >> 8) & 7;
-         int sreg_count = (inst >> 6) & 3;
+       entry_inst = inst;                      /* save for later processing */
+      else if ((inst & 0xf800) == 0x1800)      /* jal(x) */
+       cur_pc += MIPS16_INSTLEN;               /* 32-bit instruction */
+    }
 
-         /* The entry instruction always subtracts 32 from the SP.  */
-         PROC_FRAME_OFFSET(&temp_proc_desc) += 32;
+    /* The entry instruction is typically the first instruction in a function,
+       and it stores registers at offsets relative to the value of the old SP
+       (before the prologue).  But the value of the sp parameter to this
+       function is the new SP (after the prologue has been executed).  So we
+       can't calculate those offsets until we've seen the entire prologue,
+       and can calculate what the old SP must have been. */
+    if (entry_inst != 0)
+      {
+       int areg_count = (entry_inst >> 8) & 7;
+       int sreg_count = (entry_inst >> 6) & 3;
 
-         /* Check if a0-a3 were saved in the caller's argument save area.  */
-         for (reg = 4, offset = 32; reg < areg_count+4; reg++, offset += 4)
-           {
-             PROC_REG_MASK(&temp_proc_desc) |= 1 << reg;
-             temp_saved_regs.regs[reg] = sp + offset;
-           }
+       /* The entry instruction always subtracts 32 from the SP.  */
+       PROC_FRAME_OFFSET(&temp_proc_desc) += 32;
 
-         /* Check if the ra register was pushed on the stack.  */
-         offset = 28;
-         if (inst & 0x20)
-           {
-             PROC_REG_MASK(&temp_proc_desc) |= 1 << 31;
-             temp_saved_regs.regs[31] = sp + offset;
-             offset -= 4;
-           }
+       /* Now we can calculate what the SP must have been at the
+          start of the function prologue.  */
+       sp += PROC_FRAME_OFFSET(&temp_proc_desc);
 
-         /* Check if the s0 and s1 registers were pushed on the stack.  */
-         for (reg = 16; reg < sreg_count+16; reg++, offset -= 4)
-           {
-             PROC_REG_MASK(&temp_proc_desc) |= 1 << reg;
-             temp_saved_regs.regs[reg] = sp + offset;
-           }
-       }
-    }
+       /* Check if a0-a3 were saved in the caller's argument save area.  */
+       for (reg = 4, offset = 0; reg < areg_count+4; reg++)
+         {
+           PROC_REG_MASK(&temp_proc_desc) |= 1 << reg;
+           set_reg_offset (reg, sp + offset);
+           offset += MIPS_REGSIZE;
+         }
+
+       /* Check if the ra register was pushed on the stack.  */
+       offset = -4;
+       if (entry_inst & 0x20)
+         {
+           PROC_REG_MASK(&temp_proc_desc) |= 1 << RA_REGNUM;
+           set_reg_offset (RA_REGNUM, sp + offset);
+           offset -= MIPS_REGSIZE;
+         }
+
+       /* Check if the s0 and s1 registers were pushed on the stack.  */
+       for (reg = 16; reg < sreg_count+16; reg++)
+         {
+           PROC_REG_MASK(&temp_proc_desc) |= 1 << reg;
+           set_reg_offset (reg, sp + offset);
+           offset -= MIPS_REGSIZE;
+         }
+      }
 }
 
 static void
@@ -801,16 +917,14 @@ mips32_heuristic_proc_desc(start_pc, limit_pc, next_frame, sp)
   CORE_ADDR frame_addr = 0; /* Value of $r30. Used by gcc for frame-pointer */
 restart:
   PROC_FRAME_OFFSET(&temp_proc_desc) = 0;
+  PROC_FRAME_ADJUST (&temp_proc_desc) = 0;     /* offset of FP from SP */
   for (cur_pc = start_pc; cur_pc < limit_pc; cur_pc += MIPS_INSTLEN)
     {
-      char buf[MIPS_INSTLEN];
       unsigned long inst, high_word, low_word;
-      int status, reg;
+      int reg;
 
       /* Fetch the instruction.   */
-      status = (unsigned long) read_memory_nobpt (cur_pc, buf, MIPS_INSTLEN);
-      if (status) memory_error (status, cur_pc);
-      inst = (unsigned long) extract_unsigned_integer (buf, MIPS_INSTLEN);
+      inst = (unsigned long) mips_fetch_instruction (cur_pc);
 
       /* Save some code by pre-extracting some useful fields.  */
       high_word = (inst >> 16) & 0xffff;
@@ -832,7 +946,7 @@ restart:
       else if ((high_word & 0xFFE0) == 0xafa0) /* sw reg,offset($sp) */
        {
          PROC_REG_MASK(&temp_proc_desc) |= 1 << reg;
-         temp_saved_regs.regs[reg] = sp + low_word;
+         set_reg_offset (reg, sp + low_word);
        }
       else if ((high_word & 0xFFE0) == 0xffa0) /* sd reg,offset($sp) */
        {
@@ -840,12 +954,12 @@ restart:
             but the register size used is only 32 bits. Make the address
             for the saved register point to the lower 32 bits.  */
          PROC_REG_MASK(&temp_proc_desc) |= 1 << reg;
-         temp_saved_regs.regs[reg] = sp + low_word + 8 - MIPS_REGSIZE;
+         set_reg_offset (reg, sp + low_word + 8 - MIPS_REGSIZE);
        }
       else if (high_word == 0x27be)                    /* addiu $30,$sp,size */
        {
          /* Old gcc frame, r30 is virtual frame pointer.  */
-         if (low_word != PROC_FRAME_OFFSET(&temp_proc_desc))
+         if ((long)low_word != PROC_FRAME_OFFSET(&temp_proc_desc))
              frame_addr = sp + low_word;
          else if (PROC_FRAME_REG (&temp_proc_desc) == SP_REGNUM)
            {
@@ -890,7 +1004,7 @@ restart:
       else if ((high_word & 0xFFE0) == 0xafc0)         /* sw reg,offset($30) */
        {
          PROC_REG_MASK(&temp_proc_desc) |= 1 << reg;
-         temp_saved_regs.regs[reg] = frame_addr + low_word;
+         set_reg_offset (reg, frame_addr + low_word);
        }
     }
 }
@@ -911,7 +1025,7 @@ heuristic_proc_desc(start_pc, limit_pc, next_frame)
 
   if (start_pc + 200 < limit_pc)
     limit_pc = start_pc + 200;
-  if (start_pc & 1)
+  if (pc_is_mips16 (start_pc))
     mips16_heuristic_proc_desc (start_pc, limit_pc, next_frame, sp);
   else
     mips32_heuristic_proc_desc (start_pc, limit_pc, next_frame, sp);
@@ -919,16 +1033,18 @@ heuristic_proc_desc(start_pc, limit_pc, next_frame)
 }
 
 static mips_extra_func_info_t
-find_proc_desc (pc, next_frame)
+non_heuristic_proc_desc (pc, addrptr)
      CORE_ADDR pc;
-     struct frame_info *next_frame;
+     CORE_ADDR *addrptr;
 {
+  CORE_ADDR startaddr;
   mips_extra_func_info_t proc_desc;
   struct block *b = block_for_pc(pc);
   struct symbol *sym;
-  CORE_ADDR startaddr;
 
   find_pc_partial_function (pc, NULL, &startaddr, NULL);
+  if (addrptr)
+    *addrptr = startaddr;
   if (b == NULL || PC_IN_CALL_DUMMY (pc, 0, 0))
     sym = NULL;
   else
@@ -939,41 +1055,62 @@ find_proc_desc (pc, next_frame)
           symbol reading.  */
        sym = NULL;
       else
-       sym = lookup_symbol (MIPS_EFI_SYMBOL_NAME, b, LABEL_NAMESPACE,
-                            0, NULL);
+       sym = lookup_symbol (MIPS_EFI_SYMBOL_NAME, b, LABEL_NAMESPACE, 0, NULL);
     }
 
   /* If we never found a PDR for this function in symbol reading, then
      examine prologues to find the information.  */
-  if (sym && ((mips_extra_func_info_t) SYMBOL_VALUE (sym))->pdr.framereg == -1)
-    sym = NULL;
-
   if (sym)
     {
-       /* IF this is the topmost frame AND
-        * (this proc does not have debugging information OR
-        * the PC is in the procedure prologue)
-        * THEN create a "heuristic" proc_desc (by analyzing
-        * the actual code) to replace the "official" proc_desc.
-        */
-       proc_desc = (mips_extra_func_info_t) SYMBOL_VALUE (sym);
-       if (next_frame == NULL) {
-           struct symtab_and_line val;
-           struct symbol *proc_symbol =
-               PROC_DESC_IS_DUMMY(proc_desc) ? 0 : PROC_SYMBOL(proc_desc);
-
-           if (proc_symbol) {
-               val = find_pc_line (BLOCK_START
-                                   (SYMBOL_BLOCK_VALUE(proc_symbol)),
-                                   0);
-               val.pc = val.end ? val.end : pc;
+      proc_desc = (mips_extra_func_info_t) SYMBOL_VALUE (sym);
+      if (PROC_FRAME_REG (proc_desc) == -1)
+       return NULL;
+      else
+       return proc_desc;
+    }
+  else
+    return NULL;
+}
+
+
+static mips_extra_func_info_t
+find_proc_desc (pc, next_frame)
+     CORE_ADDR pc;
+     struct frame_info *next_frame;
+{
+  mips_extra_func_info_t proc_desc;
+  CORE_ADDR startaddr;
+
+  proc_desc = non_heuristic_proc_desc (pc, &startaddr);
+
+  if (proc_desc)
+    {
+      /* IF this is the topmost frame AND
+       * (this proc does not have debugging information OR
+       * the PC is in the procedure prologue)
+       * THEN create a "heuristic" proc_desc (by analyzing
+       * the actual code) to replace the "official" proc_desc.
+       */
+      if (next_frame == NULL)
+       {
+         struct symtab_and_line val;
+         struct symbol *proc_symbol =
+             PROC_DESC_IS_DUMMY(proc_desc) ? 0 : PROC_SYMBOL(proc_desc);
+
+         if (proc_symbol)
+           {
+             val = find_pc_line (BLOCK_START
+                                 (SYMBOL_BLOCK_VALUE(proc_symbol)),
+                                 0);
+             val.pc = val.end ? val.end : pc;
            }
-           if (!proc_symbol || pc < val.pc) {
-               mips_extra_func_info_t found_heuristic =
-                 heuristic_proc_desc (PROC_LOW_ADDR (proc_desc),
-                                      pc, next_frame);
-               if (found_heuristic)
-                 proc_desc = found_heuristic;
+         if (!proc_symbol || pc < val.pc)
+           {
+             mips_extra_func_info_t found_heuristic =
+               heuristic_proc_desc (PROC_LOW_ADDR (proc_desc),
+                                    pc, next_frame);
+             if (found_heuristic)
+               proc_desc = found_heuristic;
            }
        }
     }
@@ -1005,8 +1142,9 @@ get_frame_pointer(frame, proc_desc)
     struct frame_info *frame;
     mips_extra_func_info_t proc_desc;
 {
-  return ADDR_BITS_REMOVE (read_next_frame_reg (frame,
-    PROC_FRAME_REG(proc_desc)) + PROC_FRAME_OFFSET(proc_desc));
+  return ADDR_BITS_REMOVE (
+    read_next_frame_reg (frame, PROC_FRAME_REG (proc_desc)) +
+      PROC_FRAME_OFFSET (proc_desc) - PROC_FRAME_ADJUST (proc_desc));
 }
 
 mips_extra_func_info_t cached_proc_desc;
@@ -1015,29 +1153,36 @@ CORE_ADDR
 mips_frame_chain(frame)
     struct frame_info *frame;
 {
-    mips_extra_func_info_t proc_desc;
-    CORE_ADDR saved_pc = FRAME_SAVED_PC(frame);
-
-    if (saved_pc == 0 || inside_entry_file (saved_pc))
-      return 0;
-
-    proc_desc = find_proc_desc(saved_pc, frame);
-    if (!proc_desc)
-      return 0;
-
-    cached_proc_desc = proc_desc;
-
-    /* If no frame pointer and frame size is zero, we must be at end
-       of stack (or otherwise hosed).  If we don't check frame size,
-       we loop forever if we see a zero size frame.  */
-    if (PROC_FRAME_REG (proc_desc) == SP_REGNUM
-       && PROC_FRAME_OFFSET (proc_desc) == 0
-       /* The previous frame from a sigtramp frame might be frameless
-          and have frame size zero.  */
-       && !frame->signal_handler_caller)
-      return 0;
-    else
-      return get_frame_pointer (frame, proc_desc);
+  mips_extra_func_info_t proc_desc;
+  CORE_ADDR tmp;
+  CORE_ADDR saved_pc = FRAME_SAVED_PC(frame);
+
+  if (saved_pc == 0 || inside_entry_file (saved_pc))
+    return 0;
+
+  /* Check if the PC is inside a call stub.  If it is, fetch the
+     PC of the caller of that stub.  */
+  if ((tmp = mips_skip_stub (saved_pc)) != 0)
+    saved_pc = tmp;
+
+  /* Look up the procedure descriptor for this PC.  */
+  proc_desc = find_proc_desc(saved_pc, frame);
+  if (!proc_desc)
+    return 0;
+
+  cached_proc_desc = proc_desc;
+
+  /* If no frame pointer and frame size is zero, we must be at end
+     of stack (or otherwise hosed).  If we don't check frame size,
+     we loop forever if we see a zero size frame.  */
+  if (PROC_FRAME_REG (proc_desc) == SP_REGNUM
+      && PROC_FRAME_OFFSET (proc_desc) == 0
+      /* The previous frame from a sigtramp frame might be frameless
+        and have frame size zero.  */
+      && !frame->signal_handler_caller)
+    return 0;
+  else
+    return get_frame_pointer (frame, proc_desc);
 }
 
 void
@@ -1136,7 +1281,7 @@ mips_push_arguments(nargs, args, sp, struct_return, struct_addr)
   int float_argreg;
   int argnum;
   int len = 0;
-  int stack_offset;
+  int stack_offset = 0;
 
   /* Macros to round N up or down to the next A boundary; A must be
      a power of two. */
@@ -1164,11 +1309,6 @@ mips_push_arguments(nargs, args, sp, struct_return, struct_addr)
   if (struct_return)
       write_register (argreg++, struct_addr);
 
-  /* The offset onto the stack at which we will start copying parameters
-     (after the registers are used up) begins at 16 in the old ABI.
-     This leaves room for the "home" area for register parameters.  */
-  stack_offset = MIPS_EABI ? 0 : MIPS_REGSIZE * 4;
-
   /* Now load as many as possible of the first arguments into
      registers, and push the rest onto the stack.  Loop thru args
      from first to last.  */
@@ -1183,9 +1323,11 @@ mips_push_arguments(nargs, args, sp, struct_return, struct_addr)
 
       /* The EABI passes structures that do not fit in a register by
         reference. In all other cases, pass the structure by value.  */
-      if (typecode == TYPE_CODE_STRUCT && MIPS_EABI && len > MIPS_REGSIZE)
+      if (MIPS_EABI && len > MIPS_REGSIZE &&
+         (typecode == TYPE_CODE_STRUCT || typecode == TYPE_CODE_UNION))
        {
          store_address (valbuf, MIPS_REGSIZE, VALUE_ADDRESS (arg));
+         typecode = TYPE_CODE_PTR;
          len = MIPS_REGSIZE;
          val = valbuf;
        }
@@ -1201,7 +1343,12 @@ mips_push_arguments(nargs, args, sp, struct_return, struct_addr)
       /* Floating point arguments passed in registers have to be
          treated specially.  On 32-bit architectures, doubles
         are passed in register pairs; the even register gets
-        the low word, and the odd register gets the high word.  */
+        the low word, and the odd register gets the high word.
+        On non-EABI processors, the first two floating point arguments are
+        also copied to general registers, because MIPS16 functions
+        don't use float registers for arguments.  This duplication of
+        arguments in general registers can't hurt non-MIPS16 functions
+        because those registers are normally skipped.  */
       if (typecode == TYPE_CODE_FLT
          && float_argreg <= MIPS_LAST_FP_ARG_REGNUM
          && mips_fpu != MIPS_FPU_NONE)
@@ -1211,44 +1358,97 @@ mips_push_arguments(nargs, args, sp, struct_return, struct_addr)
              int low_offset = TARGET_BYTE_ORDER == BIG_ENDIAN ? 4 : 0;
              unsigned long regval;
 
+             /* Write the low word of the double to the even register(s).  */
              regval = extract_unsigned_integer (val+low_offset, 4);
-             write_register (float_argreg++, regval);  /* low word */
+             write_register (float_argreg++, regval);
+             if (!MIPS_EABI)
+               write_register (argreg+1, regval);
+
+             /* Write the high word of the double to the odd register(s).  */
              regval = extract_unsigned_integer (val+4-low_offset, 4);
-             write_register (float_argreg++, regval);  /* high word */
+             write_register (float_argreg++, regval);
+             if (!MIPS_EABI)
+               {
+                 write_register (argreg, regval);
+                 argreg += 2;
+               }
 
            }
          else
            {
+             /* This is a floating point value that fits entirely
+                in a single register.  */
              CORE_ADDR regval = extract_address (val, len);
              write_register (float_argreg++, regval);
+             if (!MIPS_EABI)
+               {
+                 write_register (argreg, regval);
+                 argreg += GDB_TARGET_IS_MIPS64 ? 1 : 2;
+               }
            }
-
-         /* If this is the old ABI, skip one or two general registers.  */
-         if (!MIPS_EABI)
-           argreg += GDB_TARGET_IS_MIPS64 ? 1 : 2;
        }
       else
        {
          /* Copy the argument to general registers or the stack in
             register-sized pieces.  Large arguments are split between
             registers and stack.  */
+         /* Note: structs whose size is not a multiple of MIPS_REGSIZE
+            are treated specially: Irix cc passes them in registers
+            where gcc sometimes puts them on the stack.  For maximum
+            compatibility, we will put them in both places.  */
+
+         int odd_sized_struct = ((len > MIPS_REGSIZE) && 
+                                 (len % MIPS_REGSIZE != 0));
          while (len > 0)
            {
              int partial_len = len < MIPS_REGSIZE ? len : MIPS_REGSIZE;
 
+             if (argreg > MIPS_LAST_ARG_REGNUM || odd_sized_struct)
+               {
+                 /* Write this portion of the argument to the stack.  */
+                 int longword_offset;
+
+                 longword_offset = 0;
+                 if (TARGET_BYTE_ORDER == BIG_ENDIAN)
+                   if (MIPS_REGSIZE == 8 &&
+                       (typecode == TYPE_CODE_INT ||
+                        typecode == TYPE_CODE_PTR ||
+                        typecode == TYPE_CODE_FLT) && len <= 4)
+                     longword_offset = 4;
+                   else if ((typecode == TYPE_CODE_STRUCT ||
+                             typecode == TYPE_CODE_UNION) &&
+                            TYPE_LENGTH (arg_type) < MIPS_REGSIZE)
+                     longword_offset = MIPS_REGSIZE - len;
+
+                 write_memory (sp + stack_offset + longword_offset, 
+                               val, partial_len);
+               }
+
+             /* Note!!! This is NOT an else clause.
+                Odd sized structs may go thru BOTH paths.  */
              if (argreg <= MIPS_LAST_ARG_REGNUM)
                {
                  CORE_ADDR regval = extract_address (val, partial_len);
 
-                 /* It's a simple argument being passed in a general
-                    register.
-                    If the argument length is smaller than the register size,
-                    we have to adjust the argument on big endian targets.
-                    But don't do this adjustment on EABI targets. */
-                 if (TARGET_BYTE_ORDER == BIG_ENDIAN
-                     && partial_len < MIPS_REGSIZE
-                     && !MIPS_EABI)
-                   regval <<= ((MIPS_REGSIZE - partial_len) * TARGET_CHAR_BIT);
+                 /* A non-floating-point argument being passed in a 
+                    general register.  If a struct or union, and if
+                    the remaining length is smaller than the register
+                    size, we have to adjust the register value on
+                    big endian targets.
+
+                    It does not seem to be necessary to do the
+                    same for integral types.
+
+                    Also don't do this adjustment on EABI targets.  */
+
+                 if (!MIPS_EABI &&
+                     TARGET_BYTE_ORDER == BIG_ENDIAN &&
+                     partial_len < MIPS_REGSIZE &&
+                     (typecode == TYPE_CODE_STRUCT ||
+                      typecode == TYPE_CODE_UNION))
+                   regval <<= ((MIPS_REGSIZE - partial_len) * 
+                               TARGET_CHAR_BIT);
+
                  write_register (argreg, regval);
                  argreg++;
     
@@ -1258,16 +1458,21 @@ mips_push_arguments(nargs, args, sp, struct_return, struct_addr)
                  if (!MIPS_EABI)
                    float_argreg = MIPS_LAST_FP_ARG_REGNUM + 1;
                }
-             else
-               {
-                 /* Write this portion of the argument to the stack.  */
-                 partial_len = len;
-                 write_memory (sp + stack_offset, val, partial_len);
-                 stack_offset += ROUND_UP (partial_len, MIPS_REGSIZE);
-               }
     
              len -= partial_len;
              val += partial_len;
+
+             /* The offset onto the stack at which we will start
+                copying parameters (after the registers are used up) 
+                begins at (4 * MIPS_REGSIZE) in the old ABI.  This 
+                leaves room for the "home" area for register parameters.
+
+                In the new EABI, the 8 register parameters do not 
+                have "home" stack space reserved for them, so the
+                stack offset does not get incremented until after
+                we have used up the 8 parameter registers.  */
+             if (!(MIPS_EABI && argnum < 8))
+               stack_offset += ROUND_UP (partial_len, MIPS_REGSIZE);
            }
        }
     }
@@ -1280,7 +1485,7 @@ mips_push_arguments(nargs, args, sp, struct_return, struct_addr)
   return sp;
 }
 
-void
+static void
 mips_push_register(CORE_ADDR *sp, int regno)
 {
   char buffer[MAX_REGISTER_RAW_SIZE];
@@ -1337,6 +1542,7 @@ mips_push_dummy_frame()
   /* Save special registers (PC, MMHI, MMLO, FPC_CSR) */
   PROC_FRAME_REG(proc_desc) = PUSH_FP_REGNUM;
   PROC_FRAME_OFFSET(proc_desc) = 0;
+  PROC_FRAME_ADJUST(proc_desc) = 0;
   mips_push_register (&sp, PC_REGNUM);
   mips_push_register (&sp, HI_REGNUM);
   mips_push_register (&sp, LO_REGNUM);
@@ -1437,22 +1643,22 @@ mips_print_register (regnum, all)
       return;
     }
 
-  /* If an even floating pointer register, also print as double. */
-  if (regnum >= FP0_REGNUM && regnum < FP0_REGNUM+MIPS_NUMREGS
+  /* If an even floating point register, also print as double. */
+  if (TYPE_CODE (REGISTER_VIRTUAL_TYPE (regnum)) == TYPE_CODE_FLT
       && !((regnum-FP0_REGNUM) & 1))
-    {
-      char dbuffer[MAX_REGISTER_RAW_SIZE]; 
+    if (REGISTER_RAW_SIZE(regnum) == 4)        /* this would be silly on MIPS64 */
+      {
+       char dbuffer[2 * MAX_REGISTER_RAW_SIZE]; 
 
-      read_relative_register_raw_bytes (regnum, dbuffer);
-      read_relative_register_raw_bytes (regnum+1, dbuffer+4); /* FIXME!! */
-#ifdef REGISTER_CONVERT_TO_TYPE
-      REGISTER_CONVERT_TO_TYPE(regnum, builtin_type_double, dbuffer);
-#endif
-      printf_filtered ("(d%d: ", regnum-FP0_REGNUM);
-      val_print (builtin_type_double, dbuffer, 0,
-                gdb_stdout, 0, 1, 0, Val_pretty_default);
-      printf_filtered ("); ");
-    }
+       read_relative_register_raw_bytes (regnum, dbuffer);
+       read_relative_register_raw_bytes (regnum+1, dbuffer+MIPS_REGSIZE);
+       REGISTER_CONVERT_TO_TYPE (regnum, builtin_type_double, dbuffer);
+
+       printf_filtered ("(d%d: ", regnum-FP0_REGNUM);
+       val_print (builtin_type_double, dbuffer, 0,
+                  gdb_stdout, 0, 1, 0, Val_pretty_default);
+       printf_filtered ("); ");
+      }
   fputs_filtered (reg_names[regnum], gdb_stdout);
 
   /* The problem with printing numeric register names (r26, etc.) is that
@@ -1466,22 +1672,146 @@ mips_print_register (regnum, all)
 
   /* If virtual format is floating, print it that way.  */
   if (TYPE_CODE (REGISTER_VIRTUAL_TYPE (regnum)) == TYPE_CODE_FLT)
-    val_print (REGISTER_VIRTUAL_TYPE (regnum), raw_buffer, 0,
-              gdb_stdout, 0, 1, 0, Val_pretty_default);
+    if (REGISTER_RAW_SIZE(regnum) == 8)
+      { /* show 8-byte floats as float AND double: */
+       int offset = 4 * (TARGET_BYTE_ORDER == BIG_ENDIAN);
+
+       printf_filtered (" (float) ");
+       val_print (builtin_type_float, raw_buffer + offset, 0,
+                  gdb_stdout, 0, 1, 0, Val_pretty_default);
+       printf_filtered (", (double) ");
+       val_print (builtin_type_double, raw_buffer, 0,
+                  gdb_stdout, 0, 1, 0, Val_pretty_default);
+      }
+    else
+      val_print (REGISTER_VIRTUAL_TYPE (regnum), raw_buffer, 0,
+                gdb_stdout, 0, 1, 0, Val_pretty_default);
   /* Else print as integer in hex.  */
   else
     print_scalar_formatted (raw_buffer, REGISTER_VIRTUAL_TYPE (regnum),
                            'x', 0, gdb_stdout);
 }
 
-/* Replacement for generic do_registers_info.  */
+/* Replacement for generic do_registers_info.  
+   Print regs in pretty columns.  */
+
+static int
+do_fp_register_row (regnum)
+     int regnum;
+{ /* do values for FP (float) regs */
+  char raw_buffer[2] [REGISTER_RAW_SIZE(FP0_REGNUM)];
+  char dbl_buffer[2 * REGISTER_RAW_SIZE(FP0_REGNUM)];
+  /* use HI and LO to control the order of combining two flt regs */
+  int HI = (TARGET_BYTE_ORDER == BIG_ENDIAN);
+  int LO = (TARGET_BYTE_ORDER != BIG_ENDIAN);
+  double doub, flt1, flt2;     /* doubles extracted from raw hex data */
+  int inv1, inv2, inv3;
+   
+  /* Get the data in raw format.  */
+  if (read_relative_register_raw_bytes (regnum, raw_buffer[HI]))
+    error ("can't read register %d (%s)", regnum, reg_names[regnum]);
+  if (REGISTER_RAW_SIZE(regnum) == 4)
+    {
+      /* 4-byte registers: we can fit two registers per row. */
+      /* Also print every pair of 4-byte regs as an 8-byte double. */
+      if (read_relative_register_raw_bytes (regnum + 1, raw_buffer[LO]))
+       error ("can't read register %d (%s)", 
+              regnum + 1, reg_names[regnum + 1]);
+
+      /* copy the two floats into one double, and unpack both */
+      memcpy (dbl_buffer, raw_buffer, sizeof(dbl_buffer));
+      flt1 = unpack_double (builtin_type_float,  raw_buffer[HI], &inv1);
+      flt2 = unpack_double (builtin_type_float,  raw_buffer[LO], &inv2);
+      doub = unpack_double (builtin_type_double, dbl_buffer,     &inv3);
+
+      printf_filtered (inv1 ? " %-5s: <invalid float>" : 
+                      " %-5s%-17.9g", reg_names[regnum],     flt1);
+      printf_filtered (inv2 ? " %-5s: <invalid float>" : 
+                      " %-5s%-17.9g", reg_names[regnum + 1], flt2);
+      printf_filtered (inv3 ? " dbl: <invalid double>\n" : 
+                      " dbl: %-24.17g\n", doub);
+      /* may want to do hex display here (future enhancement) */
+      regnum +=2;
+    }
+  else
+    { /* eight byte registers: print each one as float AND as double. */
+      int offset = 4 * (TARGET_BYTE_ORDER == BIG_ENDIAN);
+
+      memcpy (dbl_buffer, raw_buffer[HI], sizeof(dbl_buffer));
+      flt1 = unpack_double (builtin_type_float, 
+                           &raw_buffer[HI][offset], &inv1);
+      doub = unpack_double (builtin_type_double, dbl_buffer,    &inv3);
+
+      printf_filtered (inv1 ? " %-5s: <invalid float>" : 
+                      " %-5s flt: %-17.9g", reg_names[regnum], flt1);
+      printf_filtered (inv3 ? " dbl: <invalid double>\n" : 
+                      " dbl: %-24.17g\n", doub);
+      /* may want to do hex display here (future enhancement) */
+      regnum++;
+    }
+  return regnum;
+}
+
+/* Print a row's worth of GP (int) registers, with name labels above */
+
+static int
+do_gp_register_row (regnum)
+     int regnum;
+{ /* do values for GP (int) regs */
+  char raw_buffer[REGISTER_RAW_SIZE(0)];
+  int ncols = MIPS_REGSIZE == 8 ? 4 : 8;       /* display cols per row */
+  int col, byte, start_regnum = regnum;
+
+  /* For GP registers, we print a separate row of names above the vals */
+  printf_filtered ("     ");
+  for (col = 0; col < ncols && regnum < NUM_REGS; regnum++)
+    {
+      if (*reg_names[regnum] == '\0')
+       continue;       /* unused register */
+      if (TYPE_CODE (REGISTER_VIRTUAL_TYPE (regnum)) == TYPE_CODE_FLT)
+       break;  /* end the row: reached FP register */
+      printf_filtered (MIPS_REGSIZE == 8 ? "%17s" : "%9s", 
+                      reg_names[regnum]);
+      col++;
+    }
+  printf_filtered (start_regnum < MIPS_NUMREGS ? "\n R%-4d" : "\n      ", 
+                  start_regnum);       /* print the R0 to R31 names */
+
+  regnum = start_regnum;       /* go back to start of row */
+  /* now print the values in hex, 4 or 8 to the row */
+  for (col = 0; col < ncols && regnum < NUM_REGS; regnum++)
+    {
+      if (*reg_names[regnum] == '\0')
+       continue;       /* unused register */
+      if (TYPE_CODE (REGISTER_VIRTUAL_TYPE (regnum)) == TYPE_CODE_FLT)
+       break;  /* end row: reached FP register */
+      /* OK: get the data in raw format.  */
+      if (read_relative_register_raw_bytes (regnum, raw_buffer))
+       error ("can't read register %d (%s)", regnum, reg_names[regnum]);
+      /* Now print the register value in hex, endian order. */
+      if (TARGET_BYTE_ORDER == BIG_ENDIAN)
+       for (byte = 0; byte < REGISTER_RAW_SIZE (regnum); byte++)
+         printf_filtered ("%02x", (unsigned char) raw_buffer[byte]);
+      else
+       for (byte = REGISTER_RAW_SIZE (regnum) - 1; byte >= 0; byte--)
+         printf_filtered ("%02x", (unsigned char) raw_buffer[byte]);
+      printf_filtered (" ");
+      col++;
+    }
+  if (col > 0) /* ie. if we actually printed anything... */
+    printf_filtered ("\n");
+
+  return regnum;
+}
+
+/* MIPS_DO_REGISTERS_INFO(): called by "info register" command */
 
 void
 mips_do_registers_info (regnum, fpregs)
      int regnum;
      int fpregs;
 {
-  if (regnum != -1)
+  if (regnum != -1)    /* do one specified register */
     {
       if (*(reg_names[regnum]) == '\0')
        error ("Not a valid register for the current processor type");
@@ -1489,30 +1819,17 @@ mips_do_registers_info (regnum, fpregs)
       mips_print_register (regnum, 0);
       printf_filtered ("\n");
     }
-  else
+  else                 /* do all (or most) registers */
     {
-      int did_newline = 0;
-
-      for (regnum = 0; regnum < NUM_REGS; )
-       {
-         if (((!fpregs) && regnum >= FP0_REGNUM && regnum <= FCRIR_REGNUM)
-             || *(reg_names[regnum]) == '\0')
-           {
-             regnum++;
-             continue;
-           }
-         mips_print_register (regnum, 1);
-         regnum++;
-         printf_filtered ("; ");
-         did_newline = 0;
-         if ((regnum & 3) == 0)
-           {
-             printf_filtered ("\n");
-             did_newline = 1;
-           }
-       }
-      if (!did_newline)
-       printf_filtered ("\n");
+      regnum = 0;
+      while (regnum < NUM_REGS)
+       if (TYPE_CODE(REGISTER_VIRTUAL_TYPE (regnum)) == TYPE_CODE_FLT)
+         if (fpregs)   /* true for "INFO ALL-REGISTERS" command */
+           regnum = do_fp_register_row (regnum);       /* FP regs */
+         else
+           regnum += MIPS_NUMREGS;     /* skip floating point regs */
+       else
+         regnum = do_gp_register_row (regnum);         /* GP (int) regs */
     }
 }
 
@@ -1559,7 +1876,7 @@ mips_step_skips_delay (pc)
   char buf[MIPS_INSTLEN];
 
   /* There is no branch delay slot on MIPS16.  */
-  if (pc & 1)
+  if (pc_is_mips16 (pc))
     return 0;
 
   if (target_read_memory (pc, buf, MIPS_INSTLEN) != 0)
@@ -1587,14 +1904,9 @@ mips32_skip_prologue (pc, lenient)
        or in the gcc frame.  */
     for (end_pc = pc + 100; pc < end_pc; pc += MIPS_INSTLEN)
       {
-       char buf[MIPS_INSTLEN];
-       int status;
        unsigned long high_word;
 
-       status = read_memory_nobpt (pc, buf, MIPS_INSTLEN);
-       if (status)
-         memory_error (status, pc);
-       inst = (unsigned long)extract_unsigned_integer (buf, MIPS_INSTLEN);
+       inst = mips_fetch_instruction (pc);
        high_word = (inst >> 16) & 0xffff;
 
 #if 0
@@ -1679,6 +1991,8 @@ mips16_skip_prologue (pc, lenient)
      int lenient;
 {
     CORE_ADDR end_pc;
+    int extend_bytes = 0;
+    int prev_extend_bytes;
 
     /* Table of instructions likely to be found in a function prologue.  */
     static struct
@@ -1697,6 +2011,7 @@ mips16_skip_prologue (pc, lenient)
       { 0xd980, 0xff80 },      /* sw $a0-$a3,n($s1) */
       { 0x6704, 0xff1c },      /* move reg,$a0-$a3 */
       { 0xe809, 0xf81f },      /* entry pseudo-op */
+      { 0x0100, 0xff00 },      /* addiu $s1,$sp,n */
       { 0, 0 }                 /* end of table marker */
     };
 
@@ -1705,22 +2020,10 @@ mips16_skip_prologue (pc, lenient)
        or in the gcc frame.  */
     for (end_pc = pc + 100; pc < end_pc; pc += MIPS16_INSTLEN)
       {
-       char buf[MIPS16_INSTLEN];
-       int status;
        unsigned short inst;
-       int extend_bytes;
-       int prev_extend_bytes;
        int i;
 
-       status = read_memory_nobpt (pc & ~1, buf, MIPS16_INSTLEN);
-       if (status)
-         memory_error (status, pc);
-       inst = (unsigned long)extract_unsigned_integer (buf, MIPS16_INSTLEN);
-
-#if 0
-       if (lenient && is_delayed (inst))
-         continue;
-#endif
+       inst = mips_fetch_instruction (pc);
 
        /* Normally we ignore an extend instruction.  However, if it is
           not followed by a valid prologue instruction, we must adjust
@@ -1739,7 +2042,7 @@ mips16_skip_prologue (pc, lenient)
          if ((inst & table[i].mask) == table[i].inst)  /* found, get out */
            break;
        if (table[i].mask != 0)                 /* it was in table? */
-         continue;                             /* ignore it
+         continue;                             /* ignore it */
        else                                    /* non-prologue */
          {
            /* Return the current pc, adjusted backwards by 2 if
@@ -1747,6 +2050,7 @@ mips16_skip_prologue (pc, lenient)
            return pc - prev_extend_bytes;
          }
     }
+  return pc;
 }
 
 /* To skip prologues, I use this predicate.  Returns either PC itself
@@ -1775,7 +2079,7 @@ mips_skip_prologue (pc, lenient)
   /* Can't determine prologue from the symbol table, need to examine
      instructions.  */
 
-  if (pc & 1)
+  if (pc_is_mips16 (pc))
     return mips16_skip_prologue (pc, lenient);
   else
     return mips32_skip_prologue (pc, lenient);
@@ -1809,23 +2113,26 @@ mips_extract_return_value (valtype, regbuf, valbuf)
 {
   int regnum;
   int offset = 0;
+  int len = TYPE_LENGTH (valtype);
   
   regnum = 2;
   if (TYPE_CODE (valtype) == TYPE_CODE_FLT
-       && (mips_fpu == MIPS_FPU_DOUBLE
-          || (mips_fpu == MIPS_FPU_SINGLE && TYPE_LENGTH (valtype) <= 4))) /* FIXME!! */
+      && (mips_fpu == MIPS_FPU_DOUBLE
+         || (mips_fpu == MIPS_FPU_SINGLE && len <= MIPS_REGSIZE)))
     regnum = FP0_REGNUM;
 
-  if (TARGET_BYTE_ORDER == BIG_ENDIAN
-      && TYPE_CODE (valtype) != TYPE_CODE_FLT
-      && TYPE_LENGTH (valtype) < REGISTER_RAW_SIZE (regnum))
-    offset = REGISTER_RAW_SIZE (regnum) - TYPE_LENGTH (valtype);
-
-  memcpy (valbuf, regbuf + REGISTER_BYTE (regnum) + offset,
-         TYPE_LENGTH (valtype));
-#ifdef REGISTER_CONVERT_TO_TYPE
-  REGISTER_CONVERT_TO_TYPE(regnum, valtype, valbuf);
-#endif
+  if (TARGET_BYTE_ORDER == BIG_ENDIAN)
+    { /* "un-left-justify" the value from the register */
+      if (len < REGISTER_RAW_SIZE (regnum))
+       offset = REGISTER_RAW_SIZE (regnum) - len;
+      if (len > REGISTER_RAW_SIZE (regnum)     &&      /* odd-size structs */
+         len < REGISTER_RAW_SIZE (regnum) * 2 &&
+         (TYPE_CODE (valtype) == TYPE_CODE_STRUCT ||
+          TYPE_CODE (valtype) == TYPE_CODE_UNION))
+       offset = 2 * REGISTER_RAW_SIZE (regnum) - len;
+    }
+  memcpy (valbuf, regbuf + REGISTER_BYTE (regnum) + offset, len);
+  REGISTER_CONVERT_TO_TYPE (regnum, valtype, valbuf);
 }
 
 /* Given a return value in `regbuf' with a type `valtype', 
@@ -1836,21 +2143,31 @@ mips_store_return_value (valtype, valbuf)
     char *valbuf;
 {
   int regnum;
+  int offset = 0;
+  int len = TYPE_LENGTH (valtype);
   char raw_buffer[MAX_REGISTER_RAW_SIZE];
   
   regnum = 2;
   if (TYPE_CODE (valtype) == TYPE_CODE_FLT
-       && (mips_fpu == MIPS_FPU_DOUBLE
-          || (mips_fpu == MIPS_FPU_SINGLE && TYPE_LENGTH (valtype) <= 4))) /* FIXME!! */
+      && (mips_fpu == MIPS_FPU_DOUBLE
+         || (mips_fpu == MIPS_FPU_SINGLE && len <= MIPS_REGSIZE)))
     regnum = FP0_REGNUM;
 
-  memcpy(raw_buffer, valbuf, TYPE_LENGTH (valtype));
-
-#ifdef REGISTER_CONVERT_FROM_TYPE
+  if (TARGET_BYTE_ORDER == BIG_ENDIAN)
+    { /* "left-justify" the value in the register */
+      if (len < REGISTER_RAW_SIZE (regnum))
+       offset = REGISTER_RAW_SIZE (regnum) - len;
+      if (len > REGISTER_RAW_SIZE (regnum)     &&      /* odd-size structs */
+         len < REGISTER_RAW_SIZE (regnum) * 2 &&
+         (TYPE_CODE (valtype) == TYPE_CODE_STRUCT ||
+          TYPE_CODE (valtype) == TYPE_CODE_UNION))
+       offset = 2 * REGISTER_RAW_SIZE (regnum) - len;
+    }
+  memcpy(raw_buffer + offset, valbuf, len);
   REGISTER_CONVERT_FROM_TYPE(regnum, valtype, raw_buffer);
-#endif
-
-  write_register_bytes(REGISTER_BYTE (regnum), raw_buffer, TYPE_LENGTH (valtype));
+  write_register_bytes(REGISTER_BYTE (regnum), raw_buffer, 
+                      len > REGISTER_RAW_SIZE (regnum) ? 
+                      len : REGISTER_RAW_SIZE (regnum));
 }
 
 /* Exported procedure: Is PC in the signal trampoline code */
@@ -2033,21 +2350,19 @@ gdb_print_insn_mips (memaddr, info)
      that is the start of a 16-bit function.  If we didn't do this,
      the search would fail because the symbol table says the function
      starts at an odd address, i.e. 1 byte past the given address.  */
-  proc_desc = find_proc_desc (memaddr | 1, NULL);
+  memaddr = ADDR_BITS_REMOVE (memaddr);
+  proc_desc = non_heuristic_proc_desc (MAKE_MIPS16_ADDR (memaddr), NULL);
 
   /* Make an attempt to determine if this is a 16-bit function.  If
      the procedure descriptor exists and the address therein is odd,
      it's definitely a 16-bit function.  Otherwise, we have to just
      guess that if the address passed in is odd, it's 16-bits.  */
   if (proc_desc)
-    info->mach = PROC_LOW_ADDR (proc_desc) & 1 ? 16 : 0;
+    info->mach = pc_is_mips16 (PROC_LOW_ADDR (proc_desc)) ? 16 : 0;
   else
-    info->mach = memaddr & 1 ? 16 : 0;
+    info->mach = pc_is_mips16 (memaddr) ? 16 : 0;
 
-  /* Round down the instruction address to the appropriate boundary.
-     Save the amount rounded down and subtract it from the returned size of
-     the instruction so that the next time through the address won't
-     look bogus.  */
+  /* Round down the instruction address to the appropriate boundary.  */
   memaddr &= (info->mach == 16 ? ~1 : ~3);
       
   /* Call the appropriate disassembler based on the target endian-ness.  */
@@ -2070,34 +2385,56 @@ unsigned char *mips_breakpoint_from_pc (pcptr, lenptr)
 {
   if (TARGET_BYTE_ORDER == BIG_ENDIAN)
     {
-      if (*pcptr & 1)
+      if (pc_is_mips16 (*pcptr))
        {
          static char mips16_big_breakpoint[] = MIPS16_BIG_BREAKPOINT;
-         *pcptr &= ~1;
+         *pcptr = UNMAKE_MIPS16_ADDR (*pcptr);
          *lenptr = sizeof(mips16_big_breakpoint);
          return mips16_big_breakpoint;
        }
       else
        {
          static char big_breakpoint[] = BIG_BREAKPOINT;
+         static char pmon_big_breakpoint[] = PMON_BIG_BREAKPOINT;
+         static char idt_big_breakpoint[] = IDT_BIG_BREAKPOINT;
+
          *lenptr = sizeof(big_breakpoint);
-         return big_breakpoint;
+
+         if (strcmp (target_shortname, "mips") == 0)
+           return idt_big_breakpoint;
+         else if (strcmp (target_shortname, "ddb") == 0
+                  || strcmp (target_shortname, "pmon") == 0
+                  || strcmp (target_shortname, "lsi") == 0)
+           return pmon_big_breakpoint;
+         else
+           return big_breakpoint;
        }
     }
   else
     {
-      if (*pcptr & 1)
+      if (pc_is_mips16 (*pcptr))
        {
          static char mips16_little_breakpoint[] = MIPS16_LITTLE_BREAKPOINT;
-         *pcptr &= ~1;
+         *pcptr = UNMAKE_MIPS16_ADDR (*pcptr);
          *lenptr = sizeof(mips16_little_breakpoint);
          return mips16_little_breakpoint;
        }
       else
        {
          static char little_breakpoint[] = LITTLE_BREAKPOINT;
+         static char pmon_little_breakpoint[] = PMON_LITTLE_BREAKPOINT;
+         static char idt_little_breakpoint[] = IDT_LITTLE_BREAKPOINT;
+
          *lenptr = sizeof(little_breakpoint);
-         return little_breakpoint;
+
+         if (strcmp (target_shortname, "mips") == 0)
+           return idt_little_breakpoint;
+         else if (strcmp (target_shortname, "ddb") == 0
+                  || strcmp (target_shortname, "pmon") == 0
+                  || strcmp (target_shortname, "lsi") == 0)
+           return pmon_little_breakpoint;
+         else
+           return little_breakpoint;
        }
     }
 }
@@ -2109,16 +2446,196 @@ int
 mips_about_to_return (pc)
      CORE_ADDR pc;
 {
-  if (pc & 1)
+  if (pc_is_mips16 (pc))
     /* This mips16 case isn't necessarily reliable.  Sometimes the compiler
        generates a "jr $ra"; other times it generates code to load
        the return address from the stack to an accessible register (such
        as $a3), then a "jr" using that register.  This second case
        is almost impossible to distinguish from an indirect jump
        used for switch statements, so we don't even try.  */
-    return read_memory_integer (pc & ~1, 2) == 0xe820; /* jr $ra */
+    return mips_fetch_instruction (pc) == 0xe820;      /* jr $ra */
   else
-    return read_memory_integer (pc, 4) == 0x3e00008;   /* jr $ra */
+    return mips_fetch_instruction (pc) == 0x3e00008;   /* jr $ra */
+}
+
+
+/* If PC is in a mips16 call or return stub, return the address of the target
+   PC, which is either the callee or the caller.  There are several
+   cases which must be handled:
+
+   * If the PC is in __mips16_ret_{d,s}f, this is a return stub and the
+     target PC is in $31 ($ra).
+   * If the PC is in __mips16_call_stub_{1..10}, this is a call stub
+     and the target PC is in $2.
+   * If the PC at the start of __mips16_call_stub_{s,d}f_{0..10}, i.e.
+     before the jal instruction, this is effectively a call stub
+     and the the target PC is in $2.  Otherwise this is effectively
+     a return stub and the target PC is in $18.
+
+   See the source code for the stubs in gcc/config/mips/mips16.S for
+   gory details.
+
+   This function implements the SKIP_TRAMPOLINE_CODE macro.
+*/
+
+CORE_ADDR
+mips_skip_stub (pc)
+     CORE_ADDR pc;
+{
+  char *name;
+  CORE_ADDR start_addr;
+
+  /* Find the starting address and name of the function containing the PC.  */
+  if (find_pc_partial_function (pc, &name, &start_addr, NULL) == 0)
+    return 0;
+
+  /* If the PC is in __mips16_ret_{d,s}f, this is a return stub and the
+     target PC is in $31 ($ra).  */
+  if (strcmp (name, "__mips16_ret_sf") == 0
+      || strcmp (name, "__mips16_ret_df") == 0)
+    return read_register (RA_REGNUM);
+
+  if (strncmp (name, "__mips16_call_stub_", 19) == 0)
+    {
+      /* If the PC is in __mips16_call_stub_{1..10}, this is a call stub
+         and the target PC is in $2.  */
+      if (name[19] >= '0' && name[19] <= '9')
+       return read_register (2);
+
+      /* If the PC at the start of __mips16_call_stub_{s,d}f_{0..10}, i.e.
+        before the jal instruction, this is effectively a call stub
+        and the the target PC is in $2.  Otherwise this is effectively
+        a return stub and the target PC is in $18.  */
+      else if (name[19] == 's' || name[19] == 'd')
+       {
+         if (pc == start_addr)
+           {
+             /* Check if the target of the stub is a compiler-generated
+                stub.  Such a stub for a function bar might have a name
+                like __fn_stub_bar, and might look like this:
+                     mfc1    $4,$f13
+                     mfc1    $5,$f12
+                     mfc1    $6,$f15
+                     mfc1    $7,$f14
+                     la      $1,bar   (becomes a lui/addiu pair)
+                     jr      $1
+                So scan down to the lui/addi and extract the target
+                address from those two instructions.  */
+
+             CORE_ADDR target_pc = read_register (2);
+             t_inst inst;
+             int i;
+
+             /* See if the name of the target function is  __fn_stub_*.  */
+             if (find_pc_partial_function (target_pc, &name, NULL, NULL) == 0)
+               return target_pc;
+             if (strncmp (name, "__fn_stub_", 10) != 0
+                 && strcmp (name, "etext") != 0
+                 && strcmp (name, "_etext") != 0)
+               return target_pc;
+
+             /* Scan through this _fn_stub_ code for the lui/addiu pair.
+                The limit on the search is arbitrarily set to 20
+                instructions.  FIXME.  */
+             for (i = 0, pc = 0; i < 20; i++, target_pc += MIPS_INSTLEN)
+               {
+                  inst = mips_fetch_instruction (target_pc);
+                  if ((inst & 0xffff0000) == 0x3c010000)       /* lui $at */
+                     pc = (inst << 16) & 0xffff0000;           /* high word */
+                  else if ((inst & 0xffff0000) == 0x24210000)  /* addiu $at */
+                     return pc | (inst & 0xffff);              /* low word */
+               }
+
+             /* Couldn't find the lui/addui pair, so return stub address.  */
+             return target_pc;
+           }
+         else
+           /* This is the 'return' part of a call stub.  The return
+              address is in $r18.  */
+           return read_register (18);
+       }
+    }
+  return 0;    /* not a stub */
+}
+
+
+/* Return non-zero if the PC is inside a call thunk (aka stub or trampoline).
+   This implements the IN_SOLIB_CALL_TRAMPOLINE macro.  */
+
+int
+mips_in_call_stub (pc, name)
+     CORE_ADDR pc;
+     char *name;
+{
+  CORE_ADDR start_addr;
+
+  /* Find the starting address of the function containing the PC.  If the
+     caller didn't give us a name, look it up at the same time.  */
+  if (find_pc_partial_function (pc, name ? NULL : &name, &start_addr, NULL) == 0)
+    return 0;
+
+  if (strncmp (name, "__mips16_call_stub_", 19) == 0)
+    {
+      /* If the PC is in __mips16_call_stub_{1..10}, this is a call stub.  */
+      if (name[19] >= '0' && name[19] <= '9')
+       return 1;
+      /* If the PC at the start of __mips16_call_stub_{s,d}f_{0..10}, i.e.
+        before the jal instruction, this is effectively a call stub.  */
+      else if (name[19] == 's' || name[19] == 'd')
+       return pc == start_addr;
+    }
+
+  return 0;    /* not a stub */
+}
+
+
+/* Return non-zero if the PC is inside a return thunk (aka stub or trampoline).
+   This implements the IN_SOLIB_RETURN_TRAMPOLINE macro.  */
+
+int
+mips_in_return_stub (pc, name)
+     CORE_ADDR pc;
+     char *name;
+{
+  CORE_ADDR start_addr;
+
+  /* Find the starting address of the function containing the PC.  */
+  if (find_pc_partial_function (pc, NULL, &start_addr, NULL) == 0)
+    return 0;
+
+  /* If the PC is in __mips16_ret_{d,s}f, this is a return stub.  */
+  if (strcmp (name, "__mips16_ret_sf") == 0
+      || strcmp (name, "__mips16_ret_df") == 0)
+    return 1;
+
+  /* If the PC is in __mips16_call_stub_{s,d}f_{0..10} but not at the start,
+      i.e. after the jal instruction, this is effectively a return stub.  */
+  if (strncmp (name, "__mips16_call_stub_", 19) == 0
+      && (name[19] == 's' || name[19] == 'd')
+      && pc != start_addr)
+    return 1;
+
+  return 0;    /* not a stub */
+}
+
+
+/* Return non-zero if the PC is in a library helper function that should
+   be ignored.  This implements the IGNORE_HELPER_CALL macro.  */
+
+int
+mips_ignore_helper (pc)
+     CORE_ADDR pc;
+{
+  char *name;
+
+  /* Find the starting address and name of the function containing the PC.  */
+  if (find_pc_partial_function (pc, &name, NULL, NULL) == 0)
+    return 0;
+
+  /* If the PC is in __mips16_ret_{d,s}f, this is a library helper function
+     that we want to ignore.  */
+  return (strcmp (name, "__mips16_ret_sf") == 0
+         || strcmp (name, "__mips16_ret_df") == 0);
 }
 
 
@@ -2144,8 +2661,18 @@ normal floating point support.",
   c = add_show_from_set (c, &showlist);
   c->function.sfunc = mips_show_fpu_command;
 
+#ifndef MIPS_DEFAULT_FPU_TYPE
   mips_fpu = MIPS_FPU_DOUBLE;
   mips_fpu_string = strsave ("double");
+#else
+  mips_fpu = MIPS_DEFAULT_FPU_TYPE;
+  switch (mips_fpu) 
+  {
+    case MIPS_FPU_DOUBLE:  mips_fpu_string = strsave ("double");  break;
+    case MIPS_FPU_SINGLE:  mips_fpu_string = strsave ("single");  break;
+    case MIPS_FPU_NONE:    mips_fpu_string = strsave ("none");    break;
+  }    
+#endif
 
   c = add_set_cmd ("processor", class_support, var_string_noescape,
                   (char *) &tmp_mips_processor_type,
This page took 0.069497 seconds and 4 git commands to generate.