diff options
author | Jakub Jelinek <jakub@redhat.com> | 2012-02-14 10:16:01 +0100 |
---|---|---|
committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2012-02-14 10:16:01 +0100 |
commit | 7daac345d1f14a86dfc929d622a9b881252fb14f (patch) | |
tree | c88f974ee0771104a6fb20895d87c4ff1ea6da45 /gcc | |
parent | 837487be7baafd715566395ef75b0e0f16868e7b (diff) | |
download | gcc-7daac345d1f14a86dfc929d622a9b881252fb14f.zip gcc-7daac345d1f14a86dfc929d622a9b881252fb14f.tar.gz gcc-7daac345d1f14a86dfc929d622a9b881252fb14f.tar.bz2 |
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
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/ChangeLog | 7 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/testsuite/gcc.dg/pr52210.c | 12 | ||||
-rw-r--r-- | gcc/tree-vect-slp.c | 13 |
4 files changed, 33 insertions, 4 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 0fc9120..863952c 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 38d09e5..09662d8 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 0000000..c7ebeb6 --- /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 1177816..f21abb3 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 |