Re: Fix PR middle-end/57370

2013-09-09 Thread Richard Biener
On Fri, Sep 6, 2013 at 8:47 PM, Easwaran Raman era...@google.com wrote: On Fri, Sep 6, 2013 at 12:05 AM, Richard Biener richard.guent...@gmail.com wrote: On Thu, Sep 5, 2013 at 9:23 PM, Easwaran Raman era...@google.com wrote: On Tue, Aug 27, 2013 at 3:35 AM, Richard Biener

Re: Fix PR middle-end/57370

2013-09-06 Thread Richard Biener
On Thu, Sep 5, 2013 at 9:23 PM, Easwaran Raman era...@google.com wrote: On Tue, Aug 27, 2013 at 3:35 AM, Richard Biener richard.guent...@gmail.com wrote: On Thu, Aug 1, 2013 at 1:34 AM, Easwaran Raman era...@google.com wrote: I have a new patch that supersedes this. The new patch also fixes PR

Re: Fix PR middle-end/57370

2013-09-06 Thread Easwaran Raman
On Fri, Sep 6, 2013 at 12:05 AM, Richard Biener richard.guent...@gmail.com wrote: On Thu, Sep 5, 2013 at 9:23 PM, Easwaran Raman era...@google.com wrote: On Tue, Aug 27, 2013 at 3:35 AM, Richard Biener richard.guent...@gmail.com wrote: On Thu, Aug 1, 2013 at 1:34 AM, Easwaran Raman

Re: Fix PR middle-end/57370

2013-09-05 Thread Easwaran Raman
On Tue, Aug 27, 2013 at 3:35 AM, Richard Biener richard.guent...@gmail.com wrote: On Thu, Aug 1, 2013 at 1:34 AM, Easwaran Raman era...@google.com 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

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 richard.guent...@gmail.com wrote: On Fri, Aug 30, 2013 at 6:13 PM, Easwaran Raman era...@google.com wrote: On Fri, Aug 30, 2013 at

Re: Fix PR middle-end/57370

2013-09-02 Thread Richard Biener
On Fri, Aug 30, 2013 at 6:13 PM, Easwaran Raman era...@google.com wrote: On Fri, Aug 30, 2013 at 1:25 AM, Richard Biener richard.guent...@gmail.com wrote: On Fri, Aug 30, 2013 at 2:30 AM, Easwaran Raman era...@google.com wrote: Richard, Do you want me to commit everything but the

Re: Fix PR middle-end/57370

2013-08-30 Thread Richard Biener
On Fri, Aug 30, 2013 at 2:30 AM, Easwaran Raman era...@google.com 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

Re: Fix PR middle-end/57370

2013-08-30 Thread Easwaran Raman
On Fri, Aug 30, 2013 at 1:25 AM, Richard Biener richard.guent...@gmail.com wrote: On Fri, Aug 30, 2013 at 2:30 AM, Easwaran Raman era...@google.com wrote: Richard, Do you want me to commit everything but the insert_stmt_after hunk now? Yes. There are more similar failures reported in

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 all the bugs

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

Re: Fix PR middle-end/57370

2013-08-30 Thread Easwaran Raman
On Fri, Aug 30, 2013 at 9:26 AM, Jakub Jelinek ja...@redhat.com 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

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

Re: Fix PR middle-end/57370

2013-08-27 Thread Richard Biener
On Thu, Aug 1, 2013 at 1:34 AM, Easwaran Raman era...@google.com 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-08-07 Thread Easwaran Raman
Ping. On Wed, Jul 31, 2013 at 4:34 PM, Easwaran Raman era...@google.com 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

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 era...@google.com PR middle-end/57370 * tree-ssa-reassoc.c

Re: Fix PR middle-end/57370

2013-07-12 Thread Easwaran Raman
Ping. On Thu, Jun 27, 2013 at 10:15 AM, Easwaran Raman era...@google.com 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)

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