RE: [PATCH Version 2][RFA]Improving register pressure directed hoist

2012-11-11 Thread Bin Cheng
> -Original Message- > From: Jeff Law [mailto:l...@redhat.com] > Sent: Friday, November 09, 2012 9:58 PM > To: Bin Cheng > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH Version 2][RFA]Improving register pressure directed hoist > > On 11/07/2012 02:05 PM, Bin

Re: [PATCH Version 2][RFA]Improving register pressure directed hoist

2012-11-09 Thread Jeff Law
On 11/07/2012 02:05 PM, Bin Cheng wrote: 2012-11-08 Bin Cheng * gcse.c: (struct bb_data): Add new fields, old_pressure, live_in and backup. (calculate_bb_reg_pressure): Initialize live_in and backup. (update_bb_reg_pressure): New. (should_hoist_expr_to

RE: [PATCH Version 2][RFA]Improving register pressure directed hoist

2012-11-08 Thread Bin Cheng
ving register pressure directed hoist > > > > > -Original Message- > > From: Jeff Law [mailto:l...@redhat.com] > > Sent: Tuesday, November 06, 2012 4:51 AM > > To: Bin Cheng > > Cc: gcc-patches@gcc.gnu.org > > Subject: Re: [PATCH Version 2][RFA]

RE: [PATCH Version 2][RFA]Improving register pressure directed hoist

2012-11-06 Thread Bin Cheng
> -Original Message- > From: Jeff Law [mailto:l...@redhat.com] > Sent: Tuesday, November 06, 2012 4:51 AM > To: Bin Cheng > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH Version 2][RFA]Improving register pressure directed hoist > > On 11/02/2012 0

Re: [PATCH Version 2][RFA]Improving register pressure directed hoist

2012-11-05 Thread Jeff Law
On 11/02/2012 02:34 AM, Bin Cheng wrote: Also I don't understand why the bogus patch can catch more hoist opportunities and improve code size, so please help if you have any idea about this. Well, perturbing the code, particularly in a way which is supposed to change the amount of register pre

[PATCH Version 2][RFA]Improving register pressure directed hoist

2012-11-02 Thread Bin Cheng
Hi, I posted a patch improving register pressure directed hoist at http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02552.html Turns out it has a mistake in update_bb_reg_pressure resulting in changing register pressure incorrectly. Here comes the 2nd version patch for review. Unfortunately and strange