On Sun, Jul 3, 2016 at 7:10 PM, Richard Sandiford
<rdsandif...@googlemail.com> wrote:
> Richard Biener <richard.guent...@gmail.com> writes:
>> On Wed, Jun 15, 2016 at 10:52 AM, Richard Sandiford
>> <richard.sandif...@arm.com> wrote:
>>> This is the main patch in the series.  It adds a new enum and routines
>>> for classifying a vector load or store implementation.
>>>
>>> Tested on aarch64-linux-gnu and x86_64-linux-gnu.  OK to install?
>>
>> Why's the setting and checking of the memory access type conditional on !slp?
>> I'd rather avoid doing this :/
>
> For loads we need it for hybrid SLP, since we can vectorise the
> same load twice, once for SLP and once not.  (See e.g. pr62075.c.)

Ah, indeed.

> For stores it was unnecessary cut-&-paste.
>
> Is it OK with the !slp restricted to vectorizable_load?

Yes.

Thanks,
Richard.

> Thanks,
> Richard
>

Reply via email to