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

Fix PR middle-end/57370

2013-06-27 Thread Easwaran Raman
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 the IR and caused the test program to hang. Bootstraps and no test regr