From: Jeff Law <law@redhat.com>
Date: Thu, 4 Oct 2012 20:58:30 +0000 (-0600)
Subject: re PR target/50356 (Poor if condition in h8300 config code)
X-Git-Tag: misc/gccgo-go1_1_2~408
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=358a11bba195af5bdaf0879e9e6a8d2be02fc882;p=thirdparty%2Fgcc.git

re PR target/50356 (Poor if condition in h8300 config code)

	* PR target/50356
	* config/h8300/h8300.c (h8300_rtx_costs): Fix typo in CONST_INT case.

From-SVN: r192103
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index b2b69a220984..e926489d4bf6 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2012-10-04  Jeff Law  <law@redhat.com>
+
+	* PR target/50356
+	* config/h8300/h8300.c (h8300_rtx_costs): Fix typo in CONST_INT case.
+
 2012-10-04  Jason Merrill  <jason@redhat.com>
 
 	* config/rs6000/rs6000.c (rs6000_code_end): Protect the use of
diff --git a/gcc/config/h8300/h8300.c b/gcc/config/h8300/h8300.c
index 38a922898e2a..7ec13264a11d 100644
--- a/gcc/config/h8300/h8300.c
+++ b/gcc/config/h8300/h8300.c
@@ -1,6 +1,6 @@
 /* Subroutines for insn-output.c for Renesas H8/300.
    Copyright (C) 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000,
-   2001, 2002, 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011
+   2001, 2002, 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012
    Free Software Foundation, Inc.
    Contributed by Steve Chamberlain (sac@cygnus.com),
    Jim Wilson (wilson@cygnus.com), and Doug Evans (dje@cygnus.com).
@@ -1244,7 +1244,7 @@ h8300_rtx_costs (rtx x, int code, int outer_code, int opno ATTRIBUTE_UNUSED,
 	    *total = 0;
 	    return true;
 	  }
-	if (-4 <= n || n <= 4)
+	if (-4 <= n && n <= 4)
 	  {
 	    switch ((int) n)
 	      {