diff options
author | Richard Sandiford <richard.sandiford@arm.com> | 2019-01-07 18:53:44 +0000 |
---|---|---|
committer | Richard Sandiford <rsandifo@gcc.gnu.org> | 2019-01-07 18:53:44 +0000 |
commit | cc770199ded4655756eb98399f6d26f5a0c55f29 (patch) | |
tree | 090efe1f6a373e86577ea4af3e3a903729327a64 /gcc | |
parent | 08c35030c7ece1f7034d9bf7b6d3e94138ad3c68 (diff) | |
download | gcc-cc770199ded4655756eb98399f6d26f5a0c55f29.zip gcc-cc770199ded4655756eb98399f6d26f5a0c55f29.tar.gz gcc-cc770199ded4655756eb98399f6d26f5a0c55f29.tar.bz2 |
Fix IFN_MASK_STORE handling of IFN_GOMP_SIMD_LANE
The IFN_GOMP_SIMD_LANE handling in vectorizable_store tries to use MEM_REF
offsets to maintain pointer disambiguation info. This patch makes sure
that we don't try to do the same optimisation for IFN_MASK_STOREs, which
have no similar offset argument.
The patch fixes libgomp.c-c++-common/pr66199-*.c for SVE. Previously
we had an ncopies==2 store and stored both halves to the same address.
2019-01-07 Richard Sandiford <richard.sandiford@arm.com>
gcc/
* tree-vect-stmts.c (vectorizable_store): Don't use the dataref_offset
optimization for masked stores.
From-SVN: r267654
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/tree-vect-stmts.c | 1 |
2 files changed, 6 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index c236588..c207457 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,10 @@ 2019-01-07 Richard Sandiford <richard.sandiford@arm.com> + * tree-vect-stmts.c (vectorizable_store): Don't use the dataref_offset + optimization for masked stores. + +2019-01-07 Richard Sandiford <richard.sandiford@arm.com> + PR middle-end/88567 * tree-vect-loop.c (get_initial_defs_for_reduction): Pass the output vector directly to duplicate_and_interleave instead of diff --git a/gcc/tree-vect-stmts.c b/gcc/tree-vect-stmts.c index 4555092..c0e19dd 100644 --- a/gcc/tree-vect-stmts.c +++ b/gcc/tree-vect-stmts.c @@ -7059,6 +7059,7 @@ vectorizable_store (stmt_vec_info stmt_info, gimple_stmt_iterator *gsi, bool simd_lane_access_p = STMT_VINFO_SIMD_LANE_ACCESS_P (stmt_info); if (simd_lane_access_p + && !loop_masks && TREE_CODE (DR_BASE_ADDRESS (first_dr_info->dr)) == ADDR_EXPR && VAR_P (TREE_OPERAND (DR_BASE_ADDRESS (first_dr_info->dr), 0)) && integer_zerop (DR_OFFSET (first_dr_info->dr)) |