RE: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-18 Thread Bin Cheng
> -Original Message- > From: Richard Guenther [mailto:richard.guent...@gmail.com] > Sent: Tuesday, September 18, 2012 5:50 PM > To: Bin Cheng > Cc: Bin.Cheng; gcc-patches@gcc.gnu.org; Richard Earnshaw > Subject: Re: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST

Re: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-18 Thread Richard Guenther
Tuesday, September 04, 2012 11:20 PM >> To: Richard Guenther; gcc-patches@gcc.gnu.org >> Cc: Richard Earnshaw >> Subject: Re: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold- >> const.c >> >> Sorry, I mis-sent this offline. >> >> On Tue, Se

RE: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-18 Thread Bin Cheng
Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold- > const.c > > Sorry, I mis-sent this offline. > > On Tue, Sep 4, 2012 at 11:00 PM, Bin.Cheng wrote: > >>> > >>> It's not ok (I btw fail to see the patch in this thread). The > >>> curr

Re: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-04 Thread Bin.Cheng
Sorry, I mis-sent this offline. On Tue, Sep 4, 2012 at 11:00 PM, Bin.Cheng wrote: >>> >>> It's not ok (I btw fail to see the patch in this thread). The current >>> way LOGICAL_OP_NON_SHORT_CIRCUIT is implemented/used should instead >>> be changed to always match the pattern >>> >>> LOGICAL

Re: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-04 Thread Richard Guenther
>>>>>>> -Original Message- >>>>>>> From: Richard Earnshaw >>>>>>> Sent: Thursday, July 26, 2012 9:19 PM >>>>>>> To: Andrew Pinski >>>>>>> Cc: Bin Cheng; gcc-patches@gcc.gnu.org >>&

Re: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-04 Thread Richard Earnshaw
From: Richard Earnshaw >>>>>> Sent: Thursday, July 26, 2012 9:19 PM >>>>>> To: Andrew Pinski >>>>>> Cc: Bin Cheng; gcc-patches@gcc.gnu.org >>>>>> Subject: Re: [PATCH]Remove duplicate check on BRANCH_COST in >>>>&g

Re: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-04 Thread Richard Guenther
July 26, 2012 9:19 PM >>>>> To: Andrew Pinski >>>>> Cc: Bin Cheng; gcc-patches@gcc.gnu.org >>>>> Subject: Re: [PATCH]Remove duplicate check on BRANCH_COST in >>>>> fold-const.c >>>>> >>>>> On 26/07/12 11:

Re: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-04 Thread Richard Earnshaw
Cheng; gcc-patches@gcc.gnu.org >>>> Subject: Re: [PATCH]Remove duplicate check on BRANCH_COST in >>>> fold-const.c >>>> >>>> On 26/07/12 11:27, Andrew Pinski wrote: >>>>> On Thu, Jul 26, 2012 at 3:20 AM, Bin Cheng wrote: >>>&g

Re: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-04 Thread Richard Guenther
On Tue, Sep 4, 2012 at 11:56 AM, Bin Cheng wrote: >> > -Original Message- >> > From: Richard Earnshaw >> > Sent: Thursday, July 26, 2012 9:19 PM >> > To: Andrew Pinski >> > Cc: Bin Cheng; gcc-patches@gcc.gnu.org >> > Subject: Re: [PA

Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-04 Thread Bin Cheng
> > -Original Message- > > From: Richard Earnshaw > > Sent: Thursday, July 26, 2012 9:19 PM > > To: Andrew Pinski > > Cc: Bin Cheng; gcc-patches@gcc.gnu.org > > Subject: Re: [PATCH]Remove duplicate check on BRANCH_COST in > > fold-const.c >

Ping: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-08-13 Thread Bin Cheng
Ping. > -Original Message- > From: Richard Earnshaw > Sent: Thursday, July 26, 2012 9:19 PM > To: Andrew Pinski > Cc: Bin Cheng; gcc-patches@gcc.gnu.org > Subject: Re: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c > > On 26/07/12 11:27, Andrew Pinsk

Re: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-07-26 Thread Richard Earnshaw
On 26/07/12 11:27, Andrew Pinski wrote: > On Thu, Jul 26, 2012 at 3:20 AM, Bin Cheng wrote: >> Hi, >> This patch removes the duplicate check on BRANCH_COST in fold_truth_andor. >> The BRANCH_COST condition removed is a duplicate of the default definition >> of LOGICAL_OP_NON_SHORT_CIRCUIT. >> All

Re: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-07-26 Thread Andrew Pinski
On Thu, Jul 26, 2012 at 3:20 AM, Bin Cheng wrote: > Hi, > This patch removes the duplicate check on BRANCH_COST in fold_truth_andor. > The BRANCH_COST condition removed is a duplicate of the default definition > of LOGICAL_OP_NON_SHORT_CIRCUIT. > All current targets (mips and rs6000) that provide

[PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-07-26 Thread Bin Cheng
Hi, This patch removes the duplicate check on BRANCH_COST in fold_truth_andor. The BRANCH_COST condition removed is a duplicate of the default definition of LOGICAL_OP_NON_SHORT_CIRCUIT. All current targets (mips and rs6000) that provide non-default definitions of LOGICAL_OP_SHORT_CIRCUIT set it to