Re: Fix PR middle-end/57370

2013-09-09 Thread Richard Biener
On Fri, Sep 6, 2013 at 8:47 PM, Easwaran Raman wrote: > On Fri, Sep 6, 2013 at 12:05 AM, Richard Biener > wrote: >> On Thu, Sep 5, 2013 at 9:23 PM, Easwaran Raman wrote: >>> On Tue, Aug 27, 2013 at 3:35 AM, Richard Biener >>> wrote: On Thu, Aug 1, 2013 at 1:34 AM, Easwaran Raman wrote: >>

Re: Fix PR middle-end/57370

2013-09-06 Thread Easwaran Raman
On Fri, Sep 6, 2013 at 12:05 AM, Richard Biener wrote: > On Thu, Sep 5, 2013 at 9:23 PM, Easwaran Raman wrote: >> On Tue, Aug 27, 2013 at 3:35 AM, Richard Biener >> wrote: >>> On Thu, Aug 1, 2013 at 1:34 AM, Easwaran Raman wrote: I have a new patch that supersedes this. The new patch also

Re: Fix PR middle-end/57370

2013-09-06 Thread Richard Biener
On Thu, Sep 5, 2013 at 9:23 PM, Easwaran Raman wrote: > On Tue, Aug 27, 2013 at 3:35 AM, Richard Biener > wrote: >> On Thu, Aug 1, 2013 at 1:34 AM, Easwaran Raman wrote: >>> I have a new patch that supersedes this. The new patch also fixes PR >>> tree-optimization/57393 and PR tree-optimization/

Re: Fix PR middle-end/57370

2013-09-05 Thread Easwaran Raman
On Tue, Aug 27, 2013 at 3:35 AM, Richard Biener wrote: > On Thu, Aug 1, 2013 at 1:34 AM, Easwaran Raman wrote: >> I have a new patch that supersedes this. The new patch also fixes PR >> tree-optimization/57393 and PR tree-optimization/58011. Bootstraps and >> no test regression on x86_64/linux. O

Re: Fix PR middle-end/57370

2013-09-04 Thread Easwaran Raman
Submitted the patch (r202262) without the insert_stmt_after hunk. Also fixed nits pointed out by Jakub. - Easwaran On Mon, Sep 2, 2013 at 2:31 AM, Richard Biener wrote: > On Fri, Aug 30, 2013 at 6:13 PM, Easwaran Raman wrote: >> On Fri, Aug 30, 2013 at 1:25 AM, Richard Biener >> wrote: >>> On

Re: Fix PR middle-end/57370

2013-09-02 Thread Richard Biener
On Fri, Aug 30, 2013 at 6:13 PM, Easwaran Raman wrote: > On Fri, Aug 30, 2013 at 1:25 AM, Richard Biener > wrote: >> On Fri, Aug 30, 2013 at 2:30 AM, Easwaran Raman wrote: >>> Richard, >>> Do you want me to commit everything but the insert_stmt_after hunk >>> now? >> >> Yes. >> >>> There are mo

Re: Fix PR middle-end/57370

2013-08-30 Thread Easwaran Raman
On Fri, Aug 30, 2013 at 9:26 AM, Jakub Jelinek wrote: > On Fri, Aug 30, 2013 at 09:13:34AM -0700, Easwaran Raman wrote: >> >> There are more similar failures reported in >> >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57393 and I have attached >> >> the updated patch there. Shall I send that for

Re: Fix PR middle-end/57370

2013-08-30 Thread Jakub Jelinek
On Fri, Aug 30, 2013 at 09:49:59AM -0700, Easwaran Raman wrote: > Yes, this is pretty much what I was proposing. The current > implementation doesn't rely on UIDs being unique - they only have to > be monotonically non-decreasing. So, when a UID of 0 is encountered, > go up till a non-zero UID is f

Re: Fix PR middle-end/57370

2013-08-30 Thread Jakub Jelinek
On Fri, Aug 30, 2013 at 09:13:34AM -0700, Easwaran Raman wrote: > >> There are more similar failures reported in > >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57393 and I have attached > >> the updated patch there. Shall I send that for review? Apart from the > >> debug statement issue, almost a

Re: Fix PR middle-end/57370

2013-08-30 Thread Easwaran Raman
On Fri, Aug 30, 2013 at 1:25 AM, Richard Biener wrote: > On Fri, Aug 30, 2013 at 2:30 AM, Easwaran Raman wrote: >> Richard, >> Do you want me to commit everything but the insert_stmt_after hunk >> now? > > Yes. > >> There are more similar failures reported in >> http://gcc.gnu.org/bugzilla/show_

Re: Fix PR middle-end/57370

2013-08-30 Thread Richard Biener
On Fri, Aug 30, 2013 at 2:30 AM, Easwaran Raman wrote: > Richard, > Do you want me to commit everything but the insert_stmt_after hunk > now? Yes. > There are more similar failures reported in > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57393 and I have attached > the updated patch there. Sha

Re: Fix PR middle-end/57370

2013-08-29 Thread Easwaran Raman
Richard, Do you want me to commit everything but the insert_stmt_after hunk now? There are more similar failures reported in http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57393 and I have attached the updated patch there. Shall I send that for review? Apart from the debug statement issue, almost all

Re: Fix PR middle-end/57370

2013-08-27 Thread Richard Biener
On Thu, Aug 1, 2013 at 1:34 AM, Easwaran Raman wrote: > I have a new patch that supersedes this. The new patch also fixes PR > tree-optimization/57393 and PR tree-optimization/58011. Bootstraps and > no test regression on x86_64/linux. Ok for trunk? > > 2013-07-31 Easwaran Raman > > PR midd

Re: Fix PR middle-end/57370

2013-08-07 Thread Easwaran Raman
Ping. On Wed, Jul 31, 2013 at 4:34 PM, Easwaran Raman wrote: > I have a new patch that supersedes this. The new patch also fixes PR > tree-optimization/57393 and PR tree-optimization/58011. Bootstraps and > no test regression on x86_64/linux. Ok for trunk? > > 2013-07-31 Easwaran Raman > >

Re: Fix PR middle-end/57370

2013-07-31 Thread Easwaran Raman
I have a new patch that supersedes this. The new patch also fixes PR tree-optimization/57393 and PR tree-optimization/58011. Bootstraps and no test regression on x86_64/linux. Ok for trunk? 2013-07-31 Easwaran Raman PR middle-end/57370 * tree-ssa-reassoc.c (build_and_add_sum): Fix UID

Re: Fix PR middle-end/57370

2013-07-12 Thread Easwaran Raman
Ping. On Thu, Jun 27, 2013 at 10:15 AM, Easwaran Raman wrote: > A newly generated statement in build_and_add_sum function of > tree-ssa-reassoc.c has to be assigned the UID of its adjacent > statement. In one instance, it was assigned the wrong uid (of an > earlier phi statement) which messed up