From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24568 invoked by alias); 7 Aug 2014 11:26:25 -0000 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 Received: (qmail 24558 invoked by uid 89); 7 Aug 2014 11:26:25 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,SPF_PASS autolearn=ham version=3.3.2 X-HELO: service87.mimecast.com Received: from service87.mimecast.com (HELO service87.mimecast.com) (91.220.42.44) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 07 Aug 2014 11:26:23 +0000 Received: from cam-owa1.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.21]) by service87.mimecast.com; Thu, 07 Aug 2014 12:26:21 +0100 Received: from [10.1.208.24] ([10.1.255.212]) by cam-owa1.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.3959); Thu, 7 Aug 2014 12:26:21 +0100 Message-ID: <53E3625C.8060103@arm.com> Date: Thu, 07 Aug 2014 11:26:00 -0000 From: Kyrill Tkachov User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-Version: 1.0 To: GCC Patches CC: Ramana Radhakrishnan , Richard Earnshaw Subject: [PATCH][ARM] Delete f_sels, f_seld types, use fcsel instead X-MC-Unique: 114080712262101001 Content-Type: multipart/mixed; boundary="------------040602090306080005000401" X-IsSubscribed: yes X-SW-Source: 2014-08/txt/msg00791.txt.bz2 This is a multi-part message in MIME format. --------------040602090306080005000401 Content-Type: text/plain; charset=WINDOWS-1252; format=flowed Content-Transfer-Encoding: quoted-printable Content-length: 726 Hi all, During the great types rework this must have slipped through the cracks.=20 We had two different types to describe the fcsel (A64) and vsel (A32)=20 instructions: fcsel and f_sel[s,d]. In the A53 pipeline description we use fcsel, so the AArch32 version was=20 presumably scheduled improperly This patch removes the f_sel[s,d] (I don't think we need the granularity=20 for any cores) types and gets the cmov pattern in arm.md to use the=20 fcsel like the equivalent aarch64.md pattern. Tested arm-none-eabi on an emulator. Ok for trunk? 2014-08-07 Kyrylo Tkachov * config/arm/arm.md (*cmov): Set type attribute to fcsel. * config/arm/types.md (f_sels, f_seld): Delete.= --------------040602090306080005000401 Content-Type: text/x-patch; name=arm-fcsel.patch Content-Transfer-Encoding: quoted-printable Content-Disposition: attachment; filename="arm-fcsel.patch" Content-length: 1060 diff --git a/gcc/config/arm/arm.md b/gcc/config/arm/arm.md index 6ae240e..397da96 100644 --- a/gcc/config/arm/arm.md +++ b/gcc/config/arm/arm.md @@ -7416,7 +7416,7 @@ return \"\"; }" [(set_attr "conds" "use") - (set_attr "type" "f_sel")] + (set_attr "type" "fcsel")] ) =20 (define_insn_and_split "*movsicc_insn" diff --git a/gcc/config/arm/types.md b/gcc/config/arm/types.md index efbf7a7..f4feb2d 100644 --- a/gcc/config/arm/types.md +++ b/gcc/config/arm/types.md @@ -66,7 +66,6 @@ ; f_mrc transfer vfp to arm reg. ; f_mrrc transfer vfp to two arm regs. ; f_rint[d,s] double/single floating point rount to integral. -; f_sel[d,s] double/single floating byte select. ; f_store[d,s] double/single store to memory. Used for VFP unit. ; fadd[d,s] double/single floating-point scalar addition. ; fcmp[d,s] double/single floating-point compare. @@ -571,8 +570,6 @@ f_mrrc,\ f_rintd,\ f_rints,\ - f_seld,\ - f_sels,\ f_stored,\ f_stores,\ faddd,\= --------------040602090306080005000401--