> -----Original Message----- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On > Behalf Of Bin Cheng > Sent: Wednesday, November 28, 2012 9:48 AM > To: Richard Earnshaw > Cc: gcc-patches@gcc.gnu.org > Subject: RE: [PATCH ARM] Disable "-fira-hoist-pressure" on Thumb2 > > > -----Original Message----- > > From: Richard Earnshaw > > Sent: Wednesday, November 28, 2012 2:02 AM > > To: Bin Cheng > > Cc: gcc-patches@gcc.gnu.org > > Subject: Re: [PATCH ARM] Disable "-fira-hoist-pressure" on Thumb2 > > > > 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.ch...@arm.com> > > > > > > * config/arm/arm.c (arm_option_override): Disable option > > > -fira-hoist-pressure on Thumb2. > > > > > > 2012-11-21 Bin Cheng <bin.ch...@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. > > Very sorry for missing the patch. >
Ping.
