[PATCH] remove premature vect_verify_datarefs_alignment

Kewen.Lin linkw@linux.ibm.com
Fri Jul 10 10:27:58 GMT 2020


on 2020/7/9 下午7:22, Richard Biener wrote:
> On Thu, 9 Jul 2020, Kewen.Lin wrote:
> 
>> on 2020/7/9 上午10:48, Kewen.Lin via Gcc-patches wrote:
>>> Hi Richi,
>>>
>>> on 2020/7/8 下午10:45, Richard Biener wrote:
>>>> This followup removes vect_verify_datarefs_alignment and its
>>>> premature cancellation of vectorization leaving the actual
>>>> decision whether alignment is supported to the functions
>>>> deciding whether we can vectorize a load or store.
>>>>
>>>> I'll see whether to find a suitable machine to test !hw_misalign_supported
>>>> (altivec-only ppc I think?  hints welcome...), but maybe I'm lazy...
>>>>
>>>
>>> Thanks for caring about this!  As my limited experience, Power7 machine
>>> is qualified for this even with explicit configuration -mcpu=power7,
>>
>> Oops, sorry I meant --with-cpu=power7.
>>
>>> most of vector test cases will go with -mno-allow-movmisalign there.
>>>
>>> cfarm gcc110 looks fine to use.
>>>
> 
> OK, so that showed a few testcases to adjust, thus I committed the
> following.  In particular gcc.dg/vect/slp-45.c shows we can very
> well vectorize the loops and prematurely failed to.

Thanks for testing!  It sounds nice!

BR,
Kewen


More information about the Gcc-patches mailing list