From patchwork Thu Apr 5 14:46:51 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Stubbs X-Patchwork-Id: 7681 Return-Path: X-Original-To: patchwork@peony.canonical.com Delivered-To: patchwork@peony.canonical.com Received: from fiordland.canonical.com (fiordland.canonical.com [91.189.94.145]) by peony.canonical.com (Postfix) with ESMTP id B732423E4D for ; Thu, 5 Apr 2012 14:47:04 +0000 (UTC) Received: from mail-iy0-f180.google.com (mail-iy0-f180.google.com [209.85.210.180]) by fiordland.canonical.com (Postfix) with ESMTP id 462C9A187CD for ; Thu, 5 Apr 2012 14:47:04 +0000 (UTC) Received: by iage36 with SMTP id e36so2553543iag.11 for ; Thu, 05 Apr 2012 07:47:03 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-forwarded-to:x-forwarded-for:delivered-to:received-spf:message-id :date:from:user-agent:mime-version:to:cc:subject:references :in-reply-to:content-type:x-originalarrivaltime:x-gm-message-state; bh=Svxzv3BGxlE6gynThQzPRbMkthluh9/wpIl3vpKSboE=; b=KYQ3OWmwzzmREXJ0ZHG3FRCaRqh8+UOka14tmnVb9PMVLyHjyQJC82sHRqqSzvhGik z7PBlJR9Y40OqyDoVLRk/dln0OoCeT76NxHKr4APbhlUBcvxRYGOwnCUmU/VBUQAyvFP Bv7mGNVjvQK6sjfnOYdUSjBQR12m7b1MG5mvXuaAks4Lrw0HRx47X9+MVkcC+ruwucaa 4vhL/4lDq60hK0AuL+TWskJw/SKUaz7qGMxeYaE/RoK9zIoP6SYzPvVSJncIpThmmzgN sGvQLZClNXpoN2h3KiC5iII5LX4EXks0JxKDXAGfnmIBkhEBS0QC+XsRK+iKztK3Cqgr 3JNQ== Received: by 10.50.51.197 with SMTP id m5mr2233972igo.38.1333637223567; Thu, 05 Apr 2012 07:47:03 -0700 (PDT) X-Forwarded-To: linaro-patchwork@canonical.com X-Forwarded-For: patch@linaro.org linaro-patchwork@canonical.com Delivered-To: patches@linaro.org Received: by 10.231.164.217 with SMTP id f25csp63671iby; Thu, 5 Apr 2012 07:47:02 -0700 (PDT) Received: by 10.50.182.197 with SMTP id eg5mr2248182igc.36.1333637220974; Thu, 05 Apr 2012 07:47:00 -0700 (PDT) Received: from relay1.mentorg.com (relay1.mentorg.com. [192.94.38.131]) by mx.google.com with ESMTPS id i9si4314035icv.25.2012.04.05.07.47.00 (version=TLSv1/SSLv3 cipher=OTHER); Thu, 05 Apr 2012 07:47:00 -0700 (PDT) Received-SPF: neutral (google.com: 192.94.38.131 is neither permitted nor denied by best guess record for domain of Andrew_Stubbs@mentor.com) client-ip=192.94.38.131; Authentication-Results: mx.google.com; spf=neutral (google.com: 192.94.38.131 is neither permitted nor denied by best guess record for domain of Andrew_Stubbs@mentor.com) smtp.mail=Andrew_Stubbs@mentor.com Received: from svr-orw-fem-01.mgc.mentorg.com ([147.34.98.93]) by relay1.mentorg.com with esmtp id 1SFnxi-0004Bh-Ml from Andrew_Stubbs@mentor.com ; Thu, 05 Apr 2012 07:46:58 -0700 Received: from SVR-IES-FEM-01.mgc.mentorg.com ([137.202.0.104]) by svr-orw-fem-01.mgc.mentorg.com over TLS secured channel with Microsoft SMTPSVC(6.0.3790.4675); Thu, 5 Apr 2012 07:46:58 -0700 Received: from [172.30.12.136] (137.202.0.76) by SVR-IES-FEM-01.mgc.mentorg.com (137.202.0.104) with Microsoft SMTP Server id 14.1.289.1; Thu, 5 Apr 2012 15:46:56 +0100 Message-ID: <4F7DB05B.3070800@codesourcery.com> Date: Thu, 5 Apr 2012 15:46:51 +0100 From: Andrew Stubbs User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20120329 Thunderbird/11.0.1 MIME-Version: 1.0 To: Ramana Radhakrishnan CC: Richard Henderson , Richard Earnshaw , "gcc-patches@gcc.gnu.org" , "patches@linaro.org" Subject: Re: [PATCH][ARM] NEON DImode not References: <4F4E3AA5.1070804@codesourcery.com> <4F4E67B3.8050002@arm.com> <4F4F7226.6070809@codesourcery.com> <4F4FACC2.2050508@arm.com> <4F58DBFF.6080002@codesourcery.com> <4F58F454.2030803@redhat.com> <4F7221C5.1030607@codesourcery.com> In-Reply-To: X-OriginalArrivalTime: 05 Apr 2012 14:46:58.0212 (UTC) FILETIME=[F3C07640:01CD133A] X-Gm-Message-State: ALoCoQnOVgeslXxUv/qsERFp7IbxqdBD10Oc0lut0yRTkNQbcbcZweZTbdkBy8PlmpKS8dmDuGT+ On 28/03/12 14:37, Ramana Radhakrishnan wrote: > OK if no regressions. There was a small problem with VMVN being not actually predicable. I've made the obvious change and committed the attached. Andrew 2012-04-05 Andrew Stubbs gcc/ * config/arm/arm.md (arch): Add neon_onlya8 and neon_nota8. (arch_enabled): Handle new arch types. (one_cmpldi2): Add NEON support. diff --git a/gcc/config/arm/arm.md b/gcc/config/arm/arm.md index 751997f..383c585 100644 --- a/gcc/config/arm/arm.md +++ b/gcc/config/arm/arm.md @@ -207,7 +207,7 @@ ; for ARM or Thumb-2 with arm_arch6, and nov6 for ARM without ; arm_arch6. This attribute is used to compute attribute "enabled", ; use type "any" to enable an alternative in all cases. -(define_attr "arch" "any,a,t,32,t1,t2,v6,nov6,onlya8,nota8" +(define_attr "arch" "any,a,t,32,t1,t2,v6,nov6,onlya8,neon_onlya8,nota8,neon_nota8" (const_string "any")) (define_attr "arch_enabled" "no,yes" @@ -246,8 +246,18 @@ (eq_attr "tune" "cortexa8")) (const_string "yes") + (and (eq_attr "arch" "neon_onlya8") + (eq_attr "tune" "cortexa8") + (match_test "TARGET_NEON")) + (const_string "yes") + (and (eq_attr "arch" "nota8") (not (eq_attr "tune" "cortexa8"))) + (const_string "yes") + + (and (eq_attr "arch" "neon_nota8") + (not (eq_attr "tune" "cortexa8")) + (match_test "TARGET_NEON")) (const_string "yes")] (const_string "no"))) @@ -4207,11 +4217,16 @@ "") (define_insn_and_split "one_cmpldi2" - [(set (match_operand:DI 0 "s_register_operand" "=&r,&r") - (not:DI (match_operand:DI 1 "s_register_operand" "0,r")))] + [(set (match_operand:DI 0 "s_register_operand" "=w,&r,&r,?w") + (not:DI (match_operand:DI 1 "s_register_operand" " w, 0, r, w")))] "TARGET_32BIT" - "#" - "TARGET_32BIT && reload_completed" + "@ + vmvn\t%P0, %P1 + # + # + vmvn\t%P0, %P1" + "TARGET_32BIT && reload_completed + && arm_general_register_operand (operands[0], DImode)" [(set (match_dup 0) (not:SI (match_dup 1))) (set (match_dup 2) (not:SI (match_dup 3)))] " @@ -4221,8 +4236,10 @@ operands[3] = gen_highpart (SImode, operands[1]); operands[1] = gen_lowpart (SImode, operands[1]); }" - [(set_attr "length" "8") - (set_attr "predicable" "yes")] + [(set_attr "length" "*,8,8,*") + (set_attr "predicable" "no,yes,yes,no") + (set_attr "neon_type" "neon_int_1,*,*,neon_int_1") + (set_attr "arch" "neon_nota8,*,*,neon_onlya8")] ) (define_expand "one_cmplsi2"