Re: [patch, ARM] Make 128 bits the default vector size for NEON

[Ira Rosen <ira dot rosen at linaro dot org>]

On 6 April 2011 08:22, Ira Rosen <ira.rosen@linaro.org> wrote:
>
> Added a RejectNegative, changed "illegal" to "invalid" and prepared a
> wwwdocs patch.
> The testing finished successfully, except for the test in PR
> target/48454 that fails with -mvectorize-with-neon-quad without the
> patch as well.

There is an unrelated portion of code (altivec.md) in the patch I
attached in the previous note, I am attaching the correct patch here.

Sorry,
Ira

>
> OK to apply?
>
> Thanks,
> Ira
>
> ChangeLog:
>
> ? ? * doc/invoke.texi (preferred-vector-size): Document.
> ? ? * params.h (PREFERRED_VECTOR_SIZE): Define.
> ? ? * config/arm/arm.c (arm_preferred_simd_mode): Use param
> ? ? PREFERRED_VECTOR_SIZE instead of
> ? ? TARGET_NEON_VECTORIZE_QUAD. Make 128 bits the default.
> ? ? (arm_autovectorize_vector_sizes): Likewise.
> ? ? * config/arm/arm.opt (NEON_VECTORIZE_QUAD): Add
> ? ? RejectNegative.
> ? ? * params.def (PARAM_PREFERRED_VECTOR_SIZE): Define.
>
> testsuite/ChangeLog:
>
> ? ? * lib/target-supports.exp (check_effective_target_vect_multiple_sizes):
> ? ? New procedure.
> ? ? (add_options_for_quad_vectors): Replace with ...
> ? ? (add_options_for_double_vectors): ... this.
> ? ? * gfortran.dg/vect/pr19049.f90: Expect more printings on targets that
> ? ? support multiple vector sizes since the vectorizer attempts to
> ? ? vectorize with both vector sizes.
> ? ? * gcc.dg/vect/slp-reduc-6.c, gcc.dg/vect/no-vfa-vect-79.c,
> ? ? gcc.dg/vect/no-vfa-vect-102a.c, gcc.dg/vect/vect-outer-1a.c,
> ? ? gcc.dg/vect/vect-outer-1b.c, gcc.dg/vect/vect-outer-2b.c,
> ? ? gcc.dg/vect/vect-outer-3a.c, gcc.dg/vect/no-vfa-vect-37.c,
> ? ? gcc.dg/vect/vect-outer-3b.c, gcc.dg/vect/no-vfa-vect-101.c,
> ? ? gcc.dg/vect/no-vfa-vect-102.c, gcc.dg/vect/vect-reduc-dot-s8b.c,
> ? ? gcc.dg/vect/vect-outer-1.c, gcc.dg/vect/vect-104.c: Likewise.
> ? ? * gcc.dg/vect/vect-16.c: Rename to...
> ? ? * gcc.dg/vect/no-fast-math-vect-16.c: ... this to ensure that it runs
> ? ? without -ffast-math.
> ? ? * gcc.dg/vect/vect-42.c: Run with 64 bit vectors if applicable.
> ? ? * gcc.dg/vect/vect-multitypes-6.c, gcc.dg/vect/vect-52.c,
> ? ? gcc.dg/vect/vect-54.c, gcc.dg/vect/vect-46.c, gcc.dg/vect/vect-48.c,
> ? ? gcc.dg/vect/vect-96.c, gcc.dg/vect/vect-multitypes-3.c,
> ? ? gcc.dg/vect/vect-40.c: Likewise.
> ? ? * gcc.dg/vect/vect-outer-5.c: Remove quad-vectors option as
> ? ? redundant.
> ? ? * gcc.dg/vect/vect-109.c, gcc.dg/vect/vect-peel-1.c,
> ? ? gcc.dg/vect/vect-peel-2.c, gcc.dg/vect/slp-25.c,
> ? ? gcc.dg/vect/vect-multitypes-1.c, gcc.dg/vect/slp-3.c,
> ? ? gcc.dg/vect/no-vfa-pr29145.c, gcc.dg/vect/vect-multitypes-4.c:
> ? ? Likewise.
> ? ? * gcc.dg/vect/vect.exp: Run no-fast-math-vect*.c tests with
> ? ? -fno-fast-math.
>
>
>
>>
>>
>> cheers
>> Ramana
>>
>>
>> --
>> Ramana Radhakrishnan
>>
>

Attachment: vec-size.txt
Description: Text document