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 b2b69a220984e7a09930c94b2be2ac9238bc5ba1..e926489d4bf66c3b2e21842d583d1e54e4f8bf89 100644 (file)
--- 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 38a922898e2aa23f628d463d19af044a314821ef..7ec13264a11d10b4aa44f066373997b4e7e9d69e 100644 (file)
--- 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)
              {