Hi!

On Wed, Mar 04, 2020 at 03:13:51PM +0800, Kewen.Lin wrote:
> As PR94019 shows, without misaligned vector access support but with
> realign load, the vectorized loop will end up with realign scheme.
> It generates mask (control vector) with return type vector signed
> char which breaks the not check.
> 
> The fix is to differentiate powerpc vect_hw_misalign and powerpc
> !vect_hw_misalign, permit one vector char occurance for powerpc
> !vect_hw_misalign and keep other targets same as before.
> 
> Verified it on ppc64-redhat-linux (Power7 BE).
> 
> Is it ok for trunk, and backport to GCC 9 after some burn-in time?

Why is any of this Power-specific?  Does it work on other targets if
!vect_hw_misalign?

The patch is fine everywhere as far as the rs6000 port is concerned, but
I'd like someone else who actually understands this stuff to have a look
at it as well ;-)


Segher

Reply via email to