public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [obv,fr30] fix some warnings in fr30.c
@ 2010-08-31 15:32 Nathan Froyd
  0 siblings, 0 replies; only message in thread
From: Nathan Froyd @ 2010-08-31 15:32 UTC (permalink / raw)
  To: gcc-patches

Noticed while testing other things:

gcc/config/fr30/fr30.c: In function ‘fr30_move_double’:
gcc/config/fr30/fr30.c:885: warning: ISO C90 forbids mixed declarations and code
gcc/config/fr30/fr30.c:886: warning: ISO C90 forbids mixed declarations and code
gcc/config/fr30/fr30.c:883: warning: unused variable ‘dregno’

dregno is truly dead, its last use was deleted by:

2007-11-26  Rask Ingemann Lambertsen  <rask@sygehus.dk>

	PR target/34174
	* config/fr30/fr30.c (fr30_move_double): Sanitize mem->reg case. Copy
	the address before it is clobbered.

Fixed like so.  Committed.

-Nathan

	* config/fr30/fr30.c (fr30_move_double): Delete `dregno' and extra
	semicolons.

Index: config/fr30/fr30.c
===================================================================
--- config/fr30/fr30.c	(revision 163673)
+++ config/fr30/fr30.c	(working copy)
@@ -844,9 +844,8 @@ fr30_move_double (rtx * operands)
       else if (src_code == MEM)
 	{
 	  rtx addr = XEXP (src, 0);
-	  int dregno = REGNO (dest);
-	  rtx dest0 = operand_subword (dest, 0, TRUE, mode);;
-	  rtx dest1 = operand_subword (dest, 1, TRUE, mode);;
+	  rtx dest0 = operand_subword (dest, 0, TRUE, mode);
+	  rtx dest1 = operand_subword (dest, 1, TRUE, mode);
 	  rtx new_mem;
 	  
 	  gcc_assert (GET_CODE (addr) == REG);

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2010-08-31 15:01 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-08-31 15:32 [obv,fr30] fix some warnings in fr30.c Nathan Froyd

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).