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