From: Maciej W. Rozycki <macro@linux-mips.org>
Date: Thu, 20 Dec 2007 17:13:57 +0000 (+0000)
Subject: * mips-tdep.c (mips_n32n64_push_dummy_call): Sign-extend
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=5863b5d5368ca1fb517ee065ab687d8ba2fde354;p=binutils-gdb.git

* mips-tdep.c (mips_n32n64_push_dummy_call): Sign-extend
integers and 32-bit pointers as required by the ABI.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index ce6e80d9511..3d56550ee54 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2007-12-20  David Ung  <davidu@mips.com>
+            Maciej W. Rozycki  <macro@mips.com>
+
+	* mips-tdep.c (mips_n32n64_push_dummy_call): Sign-extend
+	integers and 32-bit pointers as required by the ABI.
+
 2007-12-20  Joel Brobecker  <brobecker@adacore.com>
 
 	* symtab.h: Remove "defs.h" #include.
diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c
index 193a21b32b0..d4c8a3c197f 100644
--- a/gdb/mips-tdep.c
+++ b/gdb/mips-tdep.c
@@ -3184,8 +3184,21 @@ mips_n32n64_push_dummy_call (struct gdbarch *gdbarch, struct value *function,
 	         purpose register.  */
 	      if (argreg <= MIPS_LAST_ARG_REGNUM)
 		{
-		  LONGEST regval =
-		    extract_unsigned_integer (val, partial_len);
+		  LONGEST regval;
+
+		  /* Sign extend pointers, 32-bit integers and signed
+		     16-bit and 8-bit integers; everything else is taken
+		     as is.  */
+
+		  if ((partial_len == 4
+		       && (typecode == TYPE_CODE_PTR
+			   || typecode == TYPE_CODE_INT))
+		      || (partial_len < 4
+			  && typecode == TYPE_CODE_INT
+			  && !TYPE_UNSIGNED (arg_type)))
+		    regval = extract_signed_integer (val, partial_len);
+		  else
+		    regval = extract_unsigned_integer (val, partial_len);
 
 		  /* A non-floating-point argument being passed in a
 		     general register.  If a struct or union, and if