* mips-tdep.c (mips_eabi_push_dummy_call): Place signed, rather

	than unsigned, values in registers.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 002f3d20a515ac2342af911e66d2318ffea1b140..b14a856a932593dd019d6f94e2c54eeda72e1605 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2010-12-14  Kevin Buettner  <kevinb@redhat.com>
+
+       * mips-tdep.c (mips_eabi_push_dummy_call): Place signed, rather
+       than unsigned, values in registers.
+
 2010-12-14  Ken Werner  <ken.werner@de.ibm.com>
 
        * valops.c (value_one): Use get_array_bounds to compute the number

diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c
index 7f28c18437cc42cec3075d8b758895df60d89f4a..98e7a3e05663ee534862a504e42c75d47800b6da 100644 (file)
--- a/gdb/mips-tdep.c
+++ b/gdb/mips-tdep.c
@@ -2826,23 +2826,23 @@ mips_eabi_push_dummy_call (struct gdbarch *gdbarch, struct value *function,
            {
              int low_offset = gdbarch_byte_order (gdbarch)
                               == BFD_ENDIAN_BIG ? 4 : 0;
-             unsigned long regval;
+             long regval;
 
              /* Write the low word of the double to the even register(s).  */
-             regval = extract_unsigned_integer (val + low_offset,
-                                                4, byte_order);
+             regval = extract_signed_integer (val + low_offset,
+                                              4, byte_order);
              if (mips_debug)
                fprintf_unfiltered (gdb_stdlog, " - fpreg=%d val=%s",
                                    float_argreg, phex (regval, 4));
-             regcache_cooked_write_unsigned (regcache, float_argreg++, regval);
+             regcache_cooked_write_signed (regcache, float_argreg++, regval);
 
              /* Write the high word of the double to the odd register(s).  */
-             regval = extract_unsigned_integer (val + 4 - low_offset,
-                                                4, byte_order);
+             regval = extract_signed_integer (val + 4 - low_offset,
+                                              4, byte_order);
              if (mips_debug)
                fprintf_unfiltered (gdb_stdlog, " - fpreg=%d val=%s",
                                    float_argreg, phex (regval, 4));
-             regcache_cooked_write_unsigned (regcache, float_argreg++, regval);
+             regcache_cooked_write_signed (regcache, float_argreg++, regval);
            }
          else
            {
@@ -2850,11 +2850,11 @@ mips_eabi_push_dummy_call (struct gdbarch *gdbarch, struct value *function,
                 in a single register.  */
              /* On 32 bit ABI's the float_argreg is further adjusted
                 above to ensure that it is even register aligned.  */
-             LONGEST regval = extract_unsigned_integer (val, len, byte_order);
+             LONGEST regval = extract_signed_integer (val, len, byte_order);
              if (mips_debug)
                fprintf_unfiltered (gdb_stdlog, " - fpreg=%d val=%s",
                                    float_argreg, phex (regval, len));
-             regcache_cooked_write_unsigned (regcache, float_argreg++, regval);
+             regcache_cooked_write_signed (regcache, float_argreg++, regval);
            }
        }
       else
@@ -2937,13 +2937,13 @@ mips_eabi_push_dummy_call (struct gdbarch *gdbarch, struct value *function,
                  && !fp_register_arg_p (gdbarch, typecode, arg_type))
                {
                  LONGEST regval =
-                   extract_unsigned_integer (val, partial_len, byte_order);
+                   extract_signed_integer (val, partial_len, byte_order);
 
                  if (mips_debug)
                    fprintf_filtered (gdb_stdlog, " - reg=%d val=%s",
                                      argreg,
                                      phex (regval, regsize));
-                 regcache_cooked_write_unsigned (regcache, argreg, regval);
+                 regcache_cooked_write_signed (regcache, argreg, regval);
                  argreg++;
                }