Hi Tamar,

> This patch fixes a regression introduced by r253451.
> The target needs all three conditions to be true before it can
> vectorize unaligned accesses. This patch turns the erroneous ||
> into an &&.
>
> regtested on aarch64-none-elf, arm-none-linux-gnueabihf,
> x86_64-pc-linux-gnu, armeb-none-linux-gnueabihf and
> sparc64-unknown-linux-gnu.
>
> OK for trunk?
>
> And for the GCC-7 branch?
>
> Thanks,
> Tamar
>
> gcc/testsuite/
> 2017-10-16  Tamar Christina  <tamar.christ...@arm.com>
>
>       * gcc.dg/vect/vect-multitypes-1.c: Correct target selector.

Just a nit: we usually keep a space before and after braces in DejaGnu
statements, so

{! vect_hw_misalign } -> { ! vect_hw_misalign }

Ok for mainline with that fixed, and the gcc-7 branch after a bit of
soak time.

Thanks.
        Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University

Reply via email to