From patchwork Fri Nov 30 06:05:20 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bin Cheng X-Patchwork-Id: 202869 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) by ozlabs.org (Postfix) with SMTP id 1A2A52C007D for ; Fri, 30 Nov 2012 17:19:24 +1100 (EST) Comment: DKIM? See http://www.dkim.org DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=gcc.gnu.org; s=default; x=1354861165; h=Comment: DomainKey-Signature:Received:Received:Received:Received:Received: From:To:Cc:Subject:Date:Message-ID:MIME-Version:Content-Type: Mailing-List:Precedence:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:Sender:Delivered-To; bh=hAGSjHnn7kqR+vjhN/NQ uBEwWoY=; b=mzo3FyqovIjrol+koolTfTGZ9DzVZvo5y7b8sxcX+9dlUsfFzqFq HNNGGg43uatZivlo0jBOqUUpKsR3cKObCNbaqS2aG65WK07QpFO9NmL0+ubbTohl yClNG52VRcHp6iXJIBBHOWhAUheaSqXGv3NvkxBvRhM/SqzLTgJJvNs= Comment: DomainKeys? See http://antispam.yahoo.com/domainkeys DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=default; d=gcc.gnu.org; h=Received:Received:X-SWARE-Spam-Status:X-Spam-Check-By:Received:Received:Received:From:To:Cc:Subject:Date:Message-ID:MIME-Version:X-MC-Unique:Content-Type:X-IsSubscribed:Mailing-List:Precedence:List-Id:List-Unsubscribe:List-Archive:List-Post:List-Help:Sender:Delivered-To; b=hFHyLU9Uee2l1nZnc/1I24lzUL+177iEayWW6cfPjy3wWnz6iukBsxEPlLGYin a96yMLu8EwiCoVUVSWtvhmTk3mXeB2E9AMF9B/EbCEH+2C7S/AkBVM04Tm5z2vuJ hKaU1gEqoU0ZTuxDlSAQqVubPoxFc23jHpcIwevmEvJg0=; Received: (qmail 31747 invoked by alias); 30 Nov 2012 06:19:17 -0000 Received: (qmail 31729 invoked by uid 22791); 30 Nov 2012 06:19:16 -0000 X-SWARE-Spam-Status: No, hits=-1.3 required=5.0 tests=AWL, BAYES_00, KHOP_RCVD_UNTRUST, MSGID_MULTIPLE_AT, RCVD_IN_DNSWL_LOW 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; Fri, 30 Nov 2012 06:19:10 +0000 Received: from cam-owa1.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.21]) by service87.mimecast.com; Fri, 30 Nov 2012 06:19:08 +0000 Received: from Binsh02 ([10.1.255.212]) by cam-owa1.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.0); Fri, 30 Nov 2012 06:06:04 +0000 From: "Bin Cheng" To: Cc: "Joey Ye" Subject: [GCC ARM-Embedded-4_7/Commit]Disable -fira-hoist-pressure on Thumb2 Date: Fri, 30 Nov 2012 14:05:20 +0800 Message-ID: <003201cdcec0$af94f530$0ebedf90$@cheng@arm.com> MIME-Version: 1.0 X-MC-Unique: 112113006190806701 X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Hi, I committed the patch disabling "-fira-hoist-pressure" on Thumb2 to ARM Embedded-4_7 branch as r193979. Thanks. gcc/ChangeLog.arm 2012-11-30 Bin Cheng * config/arm/arm.c (arm_option_override): Disable option -fira-hoist-pressure on Thumb2. gcc/testsuite/ChangeLog.arm 2012-11-30 Bin Cheng * gcc.dg/hoist-register-pressure.c: Skip on ARM Thumb2. * gcc.dg/hoist-register-pressure-3.c: Ditto. Index: gcc/testsuite/ChangeLog.arm =================================================================== --- gcc/testsuite/ChangeLog.arm (revision 193978) +++ gcc/testsuite/ChangeLog.arm (revision 193979) @@ -1,3 +1,8 @@ +2012-11-30 Bin Cheng + + * gcc.dg/hoist-register-pressure.c: Skip on ARM Thumb2. + * gcc.dg/hoist-register-pressure-3.c: Ditto. + 2012-11-23 Bin Cheng Backport from mainline r193687 Index: gcc/testsuite/gcc.dg/hoist-register-pressure.c =================================================================== --- gcc/testsuite/gcc.dg/hoist-register-pressure.c (revision 193978) +++ gcc/testsuite/gcc.dg/hoist-register-pressure.c (revision 193979) @@ -1,5 +1,6 @@ /* { dg-options "-Os -fdump-rtl-hoist" } */ -/* { dg-final { scan-rtl-dump "PRE/HOIST: end of bb .* copying expression" "hoist" } } */ +/* { dg-final { scan-rtl-dump "PRE/HOIST: end of bb .* copying expression" "hoist" { target { ! arm_thumb2 } } } } */ +/* { dg-final { cleanup-rtl-dump "hoist" } } */ #define BUF 100 int a[BUF]; Index: gcc/testsuite/gcc.dg/hoist-register-pressure-3.c =================================================================== --- gcc/testsuite/gcc.dg/hoist-register-pressure-3.c (revision 193978) +++ gcc/testsuite/gcc.dg/hoist-register-pressure-3.c (revision 193979) @@ -1,5 +1,6 @@ /* { dg-options "-Os -fdump-rtl-hoist" } */ -/* { dg-final { scan-rtl-dump "PRE/HOIST: end of bb .* copying expression" "hoist" } } */ +/* { dg-final { scan-rtl-dump "PRE/HOIST: end of bb .* copying expression" "hoist" { target { ! arm_thumb2 } } } } */ +/* { dg-final { cleanup-rtl-dump "hoist" } } */ #define BUF 100 int a[BUF]; Index: gcc/ChangeLog.arm =================================================================== --- gcc/ChangeLog.arm (revision 193978) +++ gcc/ChangeLog.arm (revision 193979) @@ -1,3 +1,8 @@ +2012-11-30 Bin Cheng + + * config/arm/arm.c (arm_option_override): Disable option + -fira-hoist-pressure on Thumb2. + 2012-11-28 Bin Cheng Backport from mainline r193841 Index: gcc/config/arm/arm.c =================================================================== --- gcc/config/arm/arm.c (revision 193978) +++ gcc/config/arm/arm.c (revision 193979) @@ -2035,6 +2035,12 @@ else max_insns_skipped = current_tune->max_insns_skipped; + if (TARGET_THUMB2 && flag_ira_hoist_pressure) + { + /* Don't do register pressure directed hoist on Thumb2. */ + flag_ira_hoist_pressure = 0; + } + /* Hot/Cold partitioning is not currently supported, since we can't handle literal pool placement in that case. */ if (flag_reorder_blocks_and_partition)