From cc770199ded4655756eb98399f6d26f5a0c55f29 Mon Sep 17 00:00:00 2001 From: Richard Sandiford Date: Mon, 7 Jan 2019 18:53:44 +0000 Subject: [PATCH] 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 gcc/ * tree-vect-stmts.c (vectorizable_store): Don't use the dataref_offset optimization for masked stores. From-SVN: r267654 --- gcc/ChangeLog | 5 +++++ gcc/tree-vect-stmts.c | 1 + 2 files changed, 6 insertions(+) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index c236588c23a..c2074578d3f 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2019-01-07 Richard Sandiford + + * tree-vect-stmts.c (vectorizable_store): Don't use the dataref_offset + optimization for masked stores. + 2019-01-07 Richard Sandiford PR middle-end/88567 diff --git a/gcc/tree-vect-stmts.c b/gcc/tree-vect-stmts.c index 45550927069..c0e19dd178c 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)) -- 2.30.2