From: Jakub Jelinek <jakub@redhat.com>
Date: Tue, 14 Feb 2012 09:16:01 +0000 (+0100)
Subject: re PR tree-optimization/52210 (vect_model_simple_cost: reading uninitialised memory)
X-Git-Tag: releases/gcc-4.7.0~409
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=7daac345d1f14a86dfc929d622a9b881252fb14f;p=thirdparty%2Fgcc.git

re PR tree-optimization/52210 (vect_model_simple_cost: reading uninitialised memory)

	PR tree-optimization/52210
	* tree-vect-slp.c (vect_get_and_check_slp_defs): Call
	vect_model_simple_cost with two entry vect_def_type array instead
	of an address of dt.

	* gcc.dg/pr52210.c: New test.

From-SVN: r184201
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 0fc9120318b1..863952c343e9 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,10 @@
+2012-02-14  Jakub Jelinek  <jakub@redhat.com>
+
+	PR tree-optimization/52210
+	* tree-vect-slp.c (vect_get_and_check_slp_defs): Call
+	vect_model_simple_cost with two entry vect_def_type array instead
+	of an address of dt.
+
 2012-02-14  Richard Guenther  <rguenther@suse.de>
 
 	PR lto/52178
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 38d09e5d1328..09662d837436 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2012-02-14  Jakub Jelinek  <jakub@redhat.com>
+
+	PR tree-optimization/52210
+	* gcc.dg/pr52210.c: New test.
+
 2012-02-14  Jason Merrill  <jason@redhat.com>
 
 	PR c++/39055
diff --git a/gcc/testsuite/gcc.dg/pr52210.c b/gcc/testsuite/gcc.dg/pr52210.c
new file mode 100644
index 000000000000..c7ebeb608707
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/pr52210.c
@@ -0,0 +1,12 @@
+/* PR tree-optimization/52210 */
+/* { dg-do compile } */
+/* { dg-options "-O3" } */
+
+void
+foo (long *x, long y, long z)
+{
+  long a = x[0];
+  long b = x[1];
+  x[0] = a & ~y;
+  x[1] = b & ~z;
+}
diff --git a/gcc/tree-vect-slp.c b/gcc/tree-vect-slp.c
index 1177816ab600..f21abb38bbf9 100644
--- a/gcc/tree-vect-slp.c
+++ b/gcc/tree-vect-slp.c
@@ -321,10 +321,15 @@ vect_get_and_check_slp_defs (loop_vec_info loop_vinfo, bb_vec_info bb_vinfo,
                 vect_model_store_cost (stmt_info, ncopies_for_cost, false,
                                         dt, slp_node);
 	      else
-	        /* Not memory operation (we don't call this function for
-		   loads).  */
-		vect_model_simple_cost (stmt_info, ncopies_for_cost, &dt,
-					slp_node);
+		{
+		  enum vect_def_type dts[2];
+		  dts[0] = dt;
+		  dts[1] = vect_uninitialized_def;
+		  /* Not memory operation (we don't call this function for
+		     loads).  */
+		  vect_model_simple_cost (stmt_info, ncopies_for_cost, dts,
+					  slp_node);
+		}
 	    }
 	}
       else