This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Add a vect_get_dr_size helper function
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Richard Biener <richard dot guenther at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Sandiford <richard dot sandiford at linaro dot org>
- Date: Thu, 14 Sep 2017 16:36:04 +0200
- Subject: Re: Add a vect_get_dr_size helper function
- Authentication-results: sourceware.org; auth=none
- References: <87o9qdwmkz.fsf@linaro.org> <CAFiYyc2m1Cvwt2JXSbS3424evxWKyCS=gUwvYttMBMYbUqCMZw@mail.gmail.com> <87bmmdwf3g.fsf@linaro.org>
On Thu, Sep 14, 2017 at 4:05 PM, Richard Sandiford
<richard.sandiford@linaro.org> wrote:
> Richard Biener <richard.guenther@gmail.com> writes:
>> On Thu, Sep 14, 2017 at 1:23 PM, Richard Sandiford
>> <richard.sandiford@linaro.org> wrote:
>>> This patch adds a helper function for getting the number of
>>> bytes accessed by a scalar data reference, which helps when general
>>> modes have a variable size.
>>>
>>> Tested on aarch64-linux-gnu, x86_64-linux-gnu and powerpc64le-linux-gnu.
>>> OK to install?
>>
>> Can you put it into tree-data-ref.h?
>
> The idea (which I forgot to say) was to collect the uses within the
> vectoriser into one place so that we can assert in only one place
> that the reference is constant-sized.
>
> A general data_reference can be variable-sized, so the guarantees
> wouldn't be the same elsewhere.
>
> Would putting it in tree-vectorizer.h be OK?
Maybe name it vect_get_scalar_dr_size then?
Ok with that.
Richard.
> Thanks,
> Richard