From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27013 invoked by alias); 22 Dec 2011 13:44:08 -0000 Received: (qmail 26991 invoked by uid 22791); 22 Dec 2011 13:44:06 -0000 X-SWARE-Spam-Status: No, hits=-2.4 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,TW_VT X-Spam-Check-By: sourceware.org Received: from service87.mimecast.com (HELO service87.mimecast.com) (91.220.42.44) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 22 Dec 2011 13:43:50 +0000 Received: from cam-owa2.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.21]) by service87.mimecast.com; Thu, 22 Dec 2011 13:43:46 +0000 Received: from [10.1.69.67] ([10.1.255.212]) by cam-owa2.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.3959); Thu, 22 Dec 2011 13:43:43 +0000 Message-ID: <4EF3340F.4030603@arm.com> Date: Thu, 22 Dec 2011 13:48:00 -0000 From: Richard Earnshaw User-Agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:6.0.2) Gecko/20110902 Thunderbird/6.0.2 MIME-Version: 1.0 To: Dmitry Plotnikov CC: "gcc-patches@gcc.gnu.org" , Ira Rosen , Ramana Radhakrishnan , "Joseph S. Myers" , Richard Henderson , "dm@ispras.ru" Subject: Re: [PATCH][PING] Vectorize conversions directly References: <4EA04B20.1090009@ispras.ru> <4EA51EBB.9010506@ispras.ru> <4EAA6637.4010009@ispras.ru> <4EF32D34.90704@ispras.ru> In-Reply-To: <4EF32D34.90704@ispras.ru> X-MC-Unique: 111122213434602601 Content-Type: text/plain; charset=ISO-8859-15 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-12/txt/msg01622.txt.bz2 On 22/12/11 13:14, Dmitry Plotnikov wrote: > gcc/config/arm/ > * neon.md (float2): New. > (floatuns2): New. > (fix_trunc2): New. > (fix_truncuns2): New. > * iterators.md (V_CVTTOF): New iterator. > (V_CVTTOI): New iterator. >=20 >=20 > diff --git a/gcc/config/arm/iterators.md b/gcc/config/arm/iterators.md > index 85dd641..de4340c 100644 > --- a/gcc/config/arm/iterators.md > +++ b/gcc/config/arm/iterators.md > @@ -197,6 +197,10 @@ > (define_mode_attr V_CVTTO [(V2SI "V2SF") (V2SF "V2SI") > (V4SI "V4SF") (V4SF "V4SI")]) >=20=20 > +(define_mode_attr V_CVTTOF [(V2SI "v2sf") (V4SI "v4sf")]) > + > +(define_mode_attr V_CVTTOI [(V2SF "v2si") (V4SF "v4si")]) > + attributes can be any superset of the iterator, so you don't need two separate attributes here. ;; As above but in lower case. (define_mode_attr V_cvtto [V2SI "v2sf") (V2SF "v2si") (V4SI "v4sf") (V4SF "v4si")]) is perfectly adequate and matches other attributes in the ARM back-end. The ARM bits are OK with that change. R.