This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Use gather loads for strided accesses
- From: Jeff Law <law at redhat dot com>
- To: gcc-patches at gcc dot gnu dot org, richard dot sandiford at linaro dot org
- Date: Wed, 13 Dec 2017 09:47:40 -0700
- Subject: Re: Use gather loads for strided accesses
- Authentication-results: sourceware.org; auth=none
- References: <87efowwoly.fsf@linaro.org>
On 11/17/2017 03:02 PM, Richard Sandiford wrote:
> This patch tries to use gather loads for strided accesses,
> rather than falling back to VMAT_ELEMENTWISE.
>
> Tested on aarch64-linux-gnu (with and without SVE), x86_64-linux-gnu
> and powerpc64le-linux-gnu. OK to install?
>
> Richard
>
>
> 2017-11-17 Richard Sandiford <richard.sandiford@linaro.org>
> Alan Hayward <alan.hayward@arm.com>
> David Sherwood <david.sherwood@arm.com>
>
> gcc/
> * tree-vectorizer.h (vect_create_data_ref_ptr): Take an extra
> optional tree argument.
> * tree-vect-data-refs.c (vect_create_data_ref_ptr): Take the
> iv_step as an optional argument, but continue to use the current
> value as a fallback.
> (bump_vector_ptr): Use operand_equal_p rather than tree_int_cst_compare
> to compare the updates.
> * tree-vect-stmts.c (vect_use_strided_gather_scatters_p): New function.
> (get_load_store_type): Use it when handling a strided access.
> (vect_get_strided_load_store_ops): New function.
> (vect_get_data_ptr_increment): Likewise.
> (vectorizable_load): Handle strided gather loads. Always pass
> a step to vect_create_data_ref_ptr and bump_vector_ptr.
>
> gcc/testsuite/
> * gcc.target/aarch64/sve_strided_load_1.c: New test.
> * gcc.target/aarch64/sve_strided_load_2.c: Likewise.
> * gcc.target/aarch64/sve_strided_load_3.c: Likewise.
OK.
jeff