diff options
author | Ulrich Weigand <ulrich.weigand@linaro.org> | 2012-06-15 13:30:36 +0000 |
---|---|---|
committer | Ulrich Weigand <uweigand@gcc.gnu.org> | 2012-06-15 13:30:36 +0000 |
commit | 3ebde0e9e3028aded82176416029ef79e0423082 (patch) | |
tree | 629dfbbc2c395e48be2a51e32ab649b17842ee3c /gcc/tree-vect-data-refs.c | |
parent | 273d260f83a0d914f30f8e5a055731357d53617d (diff) | |
download | gcc-3ebde0e9e3028aded82176416029ef79e0423082.zip gcc-3ebde0e9e3028aded82176416029ef79e0423082.tar.gz gcc-3ebde0e9e3028aded82176416029ef79e0423082.tar.bz2 |
re PR tree-optimization/53636 (SLP may create invalid unaligned memory accesses)
gcc/
PR tree-optimization/53636
* tree-vect-data-refs.c (vect_compute_data_ref_alignment): Verify
stride when doing basic-block vectorization.
gcc/testsuite/
PR tree-optimization/53636
* gcc.target/arm/pr53636.c: New test.
From-SVN: r188661
Diffstat (limited to 'gcc/tree-vect-data-refs.c')
-rw-r--r-- | gcc/tree-vect-data-refs.c | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/gcc/tree-vect-data-refs.c b/gcc/tree-vect-data-refs.c index ed967f2..29dff45 100644 --- a/gcc/tree-vect-data-refs.c +++ b/gcc/tree-vect-data-refs.c @@ -861,6 +861,24 @@ vect_compute_data_ref_alignment (struct data_reference *dr) } } + /* Similarly, if we're doing basic-block vectorization, we can only use + base and misalignment information relative to an innermost loop if the + misalignment stays the same throughout the execution of the loop. + As above, this is the case if the stride of the dataref evenly divides + by the vector size. */ + if (!loop) + { + tree step = DR_STEP (dr); + HOST_WIDE_INT dr_step = TREE_INT_CST_LOW (step); + + if (dr_step % GET_MODE_SIZE (TYPE_MODE (vectype)) != 0) + { + if (vect_print_dump_info (REPORT_ALIGNMENT)) + fprintf (vect_dump, "SLP: step doesn't divide the vector-size."); + misalign = NULL_TREE; + } + } + base = build_fold_indirect_ref (base_addr); alignment = ssize_int (TYPE_ALIGN (vectype)/BITS_PER_UNIT); |