[PATCH][1/n] Dissect vectorizer GROUP_* info

[Richard Biener]

On May 23, 2018 5:29:46 PM GMT+02:00, Richard Sandiford <richard.sandiford@arm.com> wrote:
>Richard Biener <rguenther@suse.de> writes:
>> This patch splits GROUP_* into DR_GROUP_* and REDUC_GROUP_* as a step
>> towards sharing dataref and dependence analysis between vectorization
>> analysis of different vector sizes (and then comparing costs and
>choosing
>> the best one).
>>
>> The next patch will move the DR_GROUP_* fields to the DR_AUX
>vectorizer
>> data structure which means that the DR group analysis could
>eventually
>> be used from outside of the vectorizer.
>>
>> Bootstrap / regtest running on x86_64-unknown-linux-gnu.
>>
>> I'm considering applying this independently because of the churn it
>> generates to make my life easier.
>>
>> Richard - I remember you talking about patches walking in the very
>> same direction so please speak up if this disrupts your work and
>> lets coordinate.
>
>I don't think this patch clashes with what I was trying, so please
>go ahead.  I was looking at patches to remove the globalness of
>vinfo_for_stmt, mostly by avoiding it wherever possible and using
>stmt_info instead of gimple *.  That's quite a lot of churn but leaves
>very few vinfo_for_stmts left.

Ah, that's indeed nice. 

>It also included an attempt to clean up how we handle getting the
>vector versions of operands.

Ambitious ;) I've tried this many times... 

Richard. 

>Thanks,
>Richard