mips-tdep.c: Wrong signness for local holding PC register.

Addresses on MIPS are signed, and we're indeed using
regcache_cooked_read_signed to read the PC, but, we're passing it the
address of an unsigned variable, which triggers a -Wpointer-sign
warning.  I've chosen to change the variable's type.  I believe this
will end up being the same (though I can't test it).

gdb/
2013-04-19  Pedro Alves  <palves@redhat.com>

	* mips-tdep.c (mips_read_pc): Change local 'pc' type to LONGEST.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index cfd5449e02326ba47e1f8d7ab115d4c9a15737cc..9314749344faa916ed77ef835785c95f21dbd1ea 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2013-04-19  Pedro Alves  <palves@redhat.com>
+
+       * mips-tdep.c (mips_read_pc): Change local 'pc' type to LONGEST.
+
 2013-04-19  Pedro Alves  <palves@redhat.com>
 
        * mep-tdep.c (mep_get_insn): Change 'insn' parameter type to

diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c
index 8e5d5d2cd8427c0eef095b99f408b38e9af04dd9..245ffd6bb4bafc1edb1acd08faf54890d2e0811d 100644 (file)
--- a/gdb/mips-tdep.c
+++ b/gdb/mips-tdep.c
@@ -1245,7 +1245,7 @@ static CORE_ADDR
 mips_read_pc (struct regcache *regcache)
 {
   int regnum = gdbarch_pc_regnum (get_regcache_arch (regcache));
-  ULONGEST pc;
+  LONGEST pc;
 
   regcache_cooked_read_signed (regcache, regnum, &pc);
   if (is_compact_addr (pc))