From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 19933 invoked by alias); 6 May 2011 10:29:43 -0000 Received: (qmail 19917 invoked by uid 22791); 6 May 2011 10:29:42 -0000 X-SWARE-Spam-Status: No, hits=-2.4 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW X-Spam-Check-By: sourceware.org Received: from service87.mimecast.com (HELO service87.mimecast.com) (94.185.240.25) by sourceware.org (qpsmtpd/0.43rc1) with SMTP; Fri, 06 May 2011 10:29:20 +0000 Received: from cam-owa1.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.21]) by service87.mimecast.com; Fri, 06 May 2011 11:29:18 +0100 Received: from [10.1.67.34] ([10.1.255.212]) by cam-owa1.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.0); Fri, 6 May 2011 11:29:17 +0100 Subject: Re: Ping: Make 128 bits the default vector size for NEON From: Richard Earnshaw To: Ira Rosen Cc: gcc-patches@gcc.gnu.org In-Reply-To: References: Date: Fri, 06 May 2011 10:35:00 -0000 Message-Id: <1304677756.5165.16.camel@e102346-lin.cambridge.arm.com> Mime-Version: 1.0 X-MC-Unique: 111050611291802501 Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2011-05/txt/msg00500.txt.bz2 On Thu, 2011-04-21 at 09:02 +0300, Ira Rosen wrote: > http://gcc.gnu.org/ml/gcc-patches/2011-03/msg02172.html >=20 > The last version: >=20 > ChangeLog: >=20 > * 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. >=20 > testsuite/ChangeLog: >=20 > * lib/target-supports.exp (check_effective_target_vect_multiple_size= s): > 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. >=20 > Thanks, > Ira +@item preferred-vector-size +Preferred vector size in bits for targets that support multiple vector siz= es. +Invalid values are ignored. The default is 128. + Shouldn't the preferred size be the largest size supported by the target? Setting it to 128 might be OK today, but who knows what might happen in future?=20=20 R.