This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH ARM] Disable "-fira-hoist-pressure" on Thumb2

From: Richard Earnshaw <rearnsha at arm dot com>
To: Bin Cheng <bin dot cheng at arm dot com>
Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Tue, 27 Nov 2012 18:02:06 +0000
Subject: Re: [PATCH ARM] Disable "-fira-hoist-pressure" on Thumb2
References: <001d01cdcc87$6065fbc0$2131f340$@cheng@arm.com>

On 27/11/12 10:10, Bin Cheng wrote:

Hi,
I committed the patch implementing register pressure directed hoist in TRUNK
and enabled the option "-fira-hoist-pressure" for all targets by reviewer's
suggestion, although it has no obvious effect on Thumb2/x86/x86_64. Now I
collected performance data with Os and found it causes performance
regression in some cases on Thumb2, so this patch disables the option on
Thumb2 and modify the corresponding test cases.

Tested on arm-none-eabi/M3, is it OK?

Thanks.

2012-11-21 Bin Cheng <bin.cheng@arm.com>

	* config/arm/arm.c (arm_option_override): Disable option
	-fira-hoist-pressure on Thumb2.

2012-11-21 Bin Cheng <bin.cheng@arm.com>

	* gcc.dg/hoist-register-pressure-1.c: Skip on ARM Thumb2.
	* gcc.dg/hoist-register-pressure-2.c: Ditto.
	* gcc.dg/hoist-register-pressure-3.c: Ditto.

No patch.

R.

Follow-Ups:
- RE: [PATCH ARM] Disable "-fira-hoist-pressure" on Thumb2
  - From: Bin Cheng

References:
- [PATCH ARM] Disable "-fira-hoist-pressure" on Thumb2
  - From: Bin Cheng

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]