re PR middle-end/55027 (simplify vector multiplication by 1)

2012-10-29  Marc Glisse  <marc.glisse@inria.fr>

	PR middle-end/55027

gcc/
	* tree.c (real_zerop, real_onep, real_twop, real_minus_onep):
	Handle VECTOR_CST.

testsuite/
	* gcc.dg/pr55027.c: New testcase.

From-SVN: r192954

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 3f5fe29..4f14673 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2012-10-29  Marc Glisse  <marc.glisse@inria.fr>
+
+       PR middle-end/55027
+       * tree.c (real_zerop, real_onep, real_twop, real_minus_onep):
+       Handle VECTOR_CST.
+
 2012-10-29  Vladimir Makarov  <vmakarov@redhat.com>
 
        * rtlanal.c (strip_address_mutation): Use SUBREG_REG instead of

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 8878dcc..0c0124d 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2012-10-29  Marc Glisse  <marc.glisse@inria.fr>
+
+       PR middle-end/55027
+       * gcc.dg/pr55027.c: New testcase.
+
 2012-10-29  Jan Hubicka  <jh@suse.cz>
 
        * gcc.dg/ipa/inline-6.c: New testcase.

diff --git a/gcc/testsuite/gcc.dg/pr55027.c b/gcc/testsuite/gcc.dg/pr55027.c
new file mode 100644 (file)
index 0000000..e72b8fc
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/pr55027.c
@@ -0,0 +1,12 @@
+/* { dg-do compile } */
+/* { dg-options "-Ofast -fdump-tree-optimized-raw" } */
+
+typedef double v2df __attribute__ ((__vector_size__ (2 * sizeof (double))));
+
+void f (v2df *x)
+{
+  *x = 0 + 1 * *x;
+}
+
+/* { dg-final { scan-tree-dump-not "gimple_assign" "optimized" } } */
+/* { dg-final { cleanup-tree-dump "optimized" } } */

diff --git a/gcc/tree.c b/gcc/tree.c
index c3642e3..a671aed 100644 (file)
--- a/gcc/tree.c
+++ b/gcc/tree.c
@@ -1992,12 +1992,25 @@ real_zerop (const_tree expr)
 {
   STRIP_NOPS (expr);
 
-  return ((TREE_CODE (expr) == REAL_CST
-          && REAL_VALUES_EQUAL (TREE_REAL_CST (expr), dconst0)
-          && !(DECIMAL_FLOAT_MODE_P (TYPE_MODE (TREE_TYPE (expr)))))
-         || (TREE_CODE (expr) == COMPLEX_CST
-             && real_zerop (TREE_REALPART (expr))
-             && real_zerop (TREE_IMAGPART (expr))));
+  switch (TREE_CODE (expr))
+    {
+    case REAL_CST:
+      return REAL_VALUES_EQUAL (TREE_REAL_CST (expr), dconst0)
+            && !(DECIMAL_FLOAT_MODE_P (TYPE_MODE (TREE_TYPE (expr))));
+    case COMPLEX_CST:
+      return real_zerop (TREE_REALPART (expr))
+            && real_zerop (TREE_IMAGPART (expr));
+    case VECTOR_CST:
+      {
+       unsigned i;
+       for (i = 0; i < VECTOR_CST_NELTS (expr); ++i)
+         if (!real_zerop (VECTOR_CST_ELT (expr, i)))
+           return false;
+       return true;
+      }
+    default:
+      return false;
+    }
 }
 
 /* Return 1 if EXPR is the real constant one in real or complex form.
@@ -2009,12 +2022,25 @@ real_onep (const_tree expr)
 {
   STRIP_NOPS (expr);
 
-  return ((TREE_CODE (expr) == REAL_CST
-          && REAL_VALUES_EQUAL (TREE_REAL_CST (expr), dconst1)
-          && !(DECIMAL_FLOAT_MODE_P (TYPE_MODE (TREE_TYPE (expr)))))
-         || (TREE_CODE (expr) == COMPLEX_CST
-             && real_onep (TREE_REALPART (expr))
-             && real_zerop (TREE_IMAGPART (expr))));
+  switch (TREE_CODE (expr))
+    {
+    case REAL_CST:
+      return REAL_VALUES_EQUAL (TREE_REAL_CST (expr), dconst1)
+            && !(DECIMAL_FLOAT_MODE_P (TYPE_MODE (TREE_TYPE (expr))));
+    case COMPLEX_CST:
+      return real_onep (TREE_REALPART (expr))
+            && real_zerop (TREE_IMAGPART (expr));
+    case VECTOR_CST:
+      {
+       unsigned i;
+       for (i = 0; i < VECTOR_CST_NELTS (expr); ++i)
+         if (!real_onep (VECTOR_CST_ELT (expr, i)))
+           return false;
+       return true;
+      }
+    default:
+      return false;
+    }
 }
 
 /* Return 1 if EXPR is the real constant two.  Trailing zeroes matter
@@ -2025,12 +2051,25 @@ real_twop (const_tree expr)
 {
   STRIP_NOPS (expr);
 
-  return ((TREE_CODE (expr) == REAL_CST
-          && REAL_VALUES_EQUAL (TREE_REAL_CST (expr), dconst2)
-          && !(DECIMAL_FLOAT_MODE_P (TYPE_MODE (TREE_TYPE (expr)))))
-         || (TREE_CODE (expr) == COMPLEX_CST
-             && real_twop (TREE_REALPART (expr))
-             && real_zerop (TREE_IMAGPART (expr))));
+  switch (TREE_CODE (expr))
+    {
+    case REAL_CST:
+      return REAL_VALUES_EQUAL (TREE_REAL_CST (expr), dconst2)
+            && !(DECIMAL_FLOAT_MODE_P (TYPE_MODE (TREE_TYPE (expr))));
+    case COMPLEX_CST:
+      return real_twop (TREE_REALPART (expr))
+            && real_zerop (TREE_IMAGPART (expr));
+    case VECTOR_CST:
+      {
+       unsigned i;
+       for (i = 0; i < VECTOR_CST_NELTS (expr); ++i)
+         if (!real_twop (VECTOR_CST_ELT (expr, i)))
+           return false;
+       return true;
+      }
+    default:
+      return false;
+    }
 }
 
 /* Return 1 if EXPR is the real constant minus one.  Trailing zeroes
@@ -2041,12 +2080,25 @@ real_minus_onep (const_tree expr)
 {
   STRIP_NOPS (expr);
 
-  return ((TREE_CODE (expr) == REAL_CST
-          && REAL_VALUES_EQUAL (TREE_REAL_CST (expr), dconstm1)
-          && !(DECIMAL_FLOAT_MODE_P (TYPE_MODE (TREE_TYPE (expr)))))
-         || (TREE_CODE (expr) == COMPLEX_CST
-             && real_minus_onep (TREE_REALPART (expr))
-             && real_zerop (TREE_IMAGPART (expr))));
+  switch (TREE_CODE (expr))
+    {
+    case REAL_CST:
+      return REAL_VALUES_EQUAL (TREE_REAL_CST (expr), dconstm1)
+            && !(DECIMAL_FLOAT_MODE_P (TYPE_MODE (TREE_TYPE (expr))));
+    case COMPLEX_CST:
+      return real_minus_onep (TREE_REALPART (expr))
+            && real_zerop (TREE_IMAGPART (expr));
+    case VECTOR_CST:
+      {
+       unsigned i;
+       for (i = 0; i < VECTOR_CST_NELTS (expr); ++i)
+         if (!real_minus_onep (VECTOR_CST_ELT (expr, i)))
+           return false;
+       return true;
+      }
+    default:
+      return false;
+    }
 }
 
 /* Nonzero if EXP is a constant or a cast of a constant.  */