From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 43369 invoked by alias); 1 May 2015 13:07:12 -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 43346 invoked by uid 89); 1 May 2015 13:07:11 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=AWL,BAYES_00,SPF_PASS autolearn=ham version=3.3.2 X-HELO: eu-smtp-delivery-143.mimecast.com Received: from eu-smtp-delivery-143.mimecast.com (HELO eu-smtp-delivery-143.mimecast.com) (146.101.78.143) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 01 May 2015 13:07:10 +0000 Received: from cam-owa2.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.140]) by uk-mta-29.uk.mimecast.lan; Fri, 01 May 2015 14:07:07 +0100 Received: from e103246vm ([10.1.2.79]) by cam-owa2.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.3959); Fri, 1 May 2015 14:07:06 +0100 From: "Wilco Dijkstra" To: "Kyrylo Tkachov" Cc: "GCC Patches" References: <000501d05753$8d857460$a8905d20$@com> In-Reply-To: Subject: RE: [PATCH][AArch64] Fix Cortex-A53 shift costs Date: Fri, 01 May 2015 13:07:00 -0000 Message-ID: <000101d0840f$b71a89d0$254f9d70$@com> MIME-Version: 1.0 X-MC-Unique: RISfwiibSCiA9jW20maW1Q-1 Content-Type: multipart/mixed; boundary="----=_NextPart_000_0002_01D08418.18DEF1D0" X-SW-Source: 2015-05/txt/msg00044.txt.bz2 This is a multi-part message in MIME format. ------=_NextPart_000_0002_01D08418.18DEF1D0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Content-length: 756 > Marcus Shawcroft wrote: > On 5 March 2015 at 14:49, Wilco Dijkstra wrote: > > This patch fixes the shift costs for Cortex-A53 so they are more accura= te - immediate shifts > use > > SBFM/UBFM which takes 2 cycles, register controlled shifts take 1 cycle= . Bootstrap and > regression > > OK. > > > > ChangeLog: > > 2015-03-05 Wilco Dijkstra > > > > * gcc/config/arm/aarch-cost-tables.h (cortexa53_extra_costs): > > Make Cortex-A53 shift costs more accurate. >=20 > OK /Marcus Kyrill, could you check this in please? Patch attached Wilco 2015-05-01 Wilco Dijkstra * gcc/config/arm/aarch-cost-tables.h (cortexa53_extra_costs): Make Cortex-A53 shift costs more accurate. ------=_NextPart_000_0002_01D08418.18DEF1D0 Content-Type: text/plain; name=0001-Fix-A53-shift-costs.txt Content-Transfer-Encoding: quoted-printable Content-Disposition: attachment; filename="0001-Fix-A53-shift-costs.txt" Content-length: 1004 --- gcc/config/arm/aarch-cost-tables.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/gcc/config/arm/aarch-cost-tables.h b/gcc/config/arm/aarch-cost= -tables.h index 05e96a9..6bb8ede 100644 --- a/gcc/config/arm/aarch-cost-tables.h +++ b/gcc/config/arm/aarch-cost-tables.h @@ -130,12 +130,12 @@ const struct cpu_cost_table cortexa53_extra_costs =3D 0, /* arith. */ 0, /* logical. */ COSTS_N_INSNS (1), /* shift. */ - COSTS_N_INSNS (2), /* shift_reg. */ + 0, /* shift_reg. */ COSTS_N_INSNS (1), /* arith_shift. */ - COSTS_N_INSNS (2), /* arith_shift_reg. */ + COSTS_N_INSNS (1), /* arith_shift_reg. */ COSTS_N_INSNS (1), /* log_shift. */ - COSTS_N_INSNS (2), /* log_shift_reg. */ - 0, /* extend. */ + COSTS_N_INSNS (1), /* log_shift_reg. */ + COSTS_N_INSNS (1), /* extend. */ COSTS_N_INSNS (1), /* extend_arith. */ COSTS_N_INSNS (1), /* bfi. */ COSTS_N_INSNS (1), /* bfx. */ --=20 1.9.1 ------=_NextPart_000_0002_01D08418.18DEF1D0--