Re: PR tree-optimization/57337

2013-05-28 Thread Richard Biener
On Mon, May 27, 2013 at 10:20 AM, Richard Biener richard.guent...@gmail.com wrote: On Sun, May 26, 2013 at 5:53 AM, Easwaran Raman era...@google.com wrote: On Sat, May 25, 2013 at 4:46 AM, Richard Biener richard.guent...@gmail.com wrote: Easwaran Raman era...@google.com wrote: In that case,

Re: PR tree-optimization/57337

2013-05-28 Thread Easwaran Raman
On Tue, May 28, 2013 at 4:11 AM, Richard Biener richard.guent...@gmail.com wrote: On Mon, May 27, 2013 at 10:20 AM, Richard Biener richard.guent...@gmail.com wrote: On Sun, May 26, 2013 at 5:53 AM, Easwaran Raman era...@google.com wrote: On Sat, May 25, 2013 at 4:46 AM, Richard Biener

Re: PR tree-optimization/57337

2013-05-27 Thread Richard Biener
? Why not simply conservatively use gimple_uid (a) = gimple_uid (b) in not_dominated_by? Richard. Thanks, Easwaran 2013-05-23 Easwaran Raman era...@google.com PR tree-optimization/57337 * tree-ssa-reassoc.c (appears_later_in_bb): New function. (find_insert_point): Correctly identify

Re: PR tree-optimization/57337

2013-05-25 Thread Richard Biener
. Thanks, Easwaran 2013-05-23 Easwaran Raman era...@google.com PR tree-optimization/57337 * tree-ssa-reassoc.c (appears_later_in_bb): New function. (find_insert_point): Correctly identify the insertion point when two statements with the same UID is compared. Index: gcc/tree-ssa-reassoc.c

Re: PR tree-optimization/57337

2013-05-25 Thread Easwaran Raman
which statement appears earlier in a BB. Bootstraps and no test regressions in x86_64 on linux. Ok for trunk? Why not simply conservatively use gimple_uid (a) = gimple_uid (b) in not_dominated_by? Richard. Thanks, Easwaran 2013-05-23 Easwaran Raman era...@google.com PR tree

Re: PR tree-optimization/57337

2013-05-24 Thread Richard Biener
gimple_uid (a) = gimple_uid (b) in not_dominated_by? Richard. Thanks, Easwaran 2013-05-23 Easwaran Raman era...@google.com PR tree-optimization/57337 * tree-ssa-reassoc.c (appears_later_in_bb): New function. (find_insert_point): Correctly identify the insertion point when two statements

Re: PR tree-optimization/57337

2013-05-24 Thread Easwaran Raman
(a) = gimple_uid (b) in not_dominated_by? Richard. Thanks, Easwaran 2013-05-23 Easwaran Raman era...@google.com PR tree-optimization/57337 * tree-ssa-reassoc.c (appears_later_in_bb): New function. (find_insert_point): Correctly identify the insertion point when two statements

PR tree-optimization/57337

2013-05-23 Thread Easwaran Raman
This addresses the case where UID alone is not sufficient to figure out which statement appears earlier in a BB. Bootstraps and no test regressions in x86_64 on linux. Ok for trunk? Thanks, Easwaran 2013-05-23 Easwaran Raman era...@google.com PR tree-optimization/57337 * tree-ssa-reassoc.c