Patchwork i386: Fix logic error in r188785

login
register
mail settings
Submitter Richard Henderson
Date June 27, 2012, 9:36 p.m.
Message ID <1340832974-3668-1-git-send-email-rth@redhat.com>
Download mbox | patch
Permalink /patch/167748/
State New
Headers show

Comments

Richard Henderson - June 27, 2012, 9:36 p.m.
As noticed by Igor Zamyatin.  Committed.


r~
---

PR target/53749
        * config/i386/i386.c (ix86_rtx_costs): Fix typo vs UNITS_PER_WORD
        in 2012-06-23 change.  Adjust two other DImode tests as well.
---
 gcc/ChangeLog          |    6 ++++++
 gcc/config/i386/i386.c |    8 +++-----
 2 files changed, 9 insertions(+), 5 deletions(-)

Patch

diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index edfc649..aae8a4d 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -32210,9 +32210,8 @@  ix86_rtx_costs (rtx x, int code_i, int outer_code_i, int opno, int *total,
 	    }
 	  else
 	    *total = cost->fabs;
-	  return false;
 	}
-      if (GET_MODE_SIZE (mode) < UNITS_PER_WORD)
+      else if (GET_MODE_SIZE (mode) > UNITS_PER_WORD)
 	{
 	  if (CONST_INT_P (XEXP (x, 1)))
 	    {
@@ -32441,7 +32440,7 @@  ix86_rtx_costs (rtx x, int code_i, int outer_code_i, int opno, int *total,
     case AND:
     case IOR:
     case XOR:
-      if (!TARGET_64BIT && mode == DImode)
+      if (GET_MODE_SIZE (mode) > UNITS_PER_WORD)
 	{
 	  *total = (cost->add * 2
 		    + (rtx_cost (XEXP (x, 0), outer_code, opno, speed)
@@ -32479,9 +32478,8 @@  ix86_rtx_costs (rtx x, int code_i, int outer_code_i, int opno, int *total,
 	  /* At least for published AMD latencies, this really is the same
 	     as the latency for a simple fpu operation like fabs.  */
 	  *total = cost->fabs;
-	  return false;
 	}
-      if (!TARGET_64BIT && mode == DImode)
+      else if (GET_MODE_SIZE (mode) > UNITS_PER_WORD)
 	*total = cost->add * 2;
       else
 	*total = cost->add;