Re: [PATCH] Fix PR58775

2013-10-21 Thread Jakub Jelinek
On Mon, Oct 21, 2013 at 01:20:06PM +0800, Zhenqiang Chen wrote: > Thanks for the comments. > > Patch is updated to set uid in update_range_test after > force_gimple_operand_gsi. > > I am not sure the patch can cover all cases. If not, I think > force_gimple_operand_gsi_1 maybe the only place to

Re: [PATCH] Fix PR58775

2013-10-20 Thread Zhenqiang Chen
On 18 October 2013 19:09, Jakub Jelinek wrote: > On Fri, Oct 18, 2013 at 06:36:44PM +0800, Zhenqiang Chen wrote: > --- a/gcc/tree-ssa-reassoc.c > +++ b/gcc/tree-ssa-reassoc.c > @@ -2861,6 +2861,19 @@ swap_ops_for_binary_stmt (vec ops, > } > } > > +/* Determine if stmt A is in th next list of

Re: [PATCH] Fix PR58775

2013-10-18 Thread Jakub Jelinek
On Fri, Oct 18, 2013 at 06:36:44PM +0800, Zhenqiang Chen wrote: --- a/gcc/tree-ssa-reassoc.c +++ b/gcc/tree-ssa-reassoc.c @@ -2861,6 +2861,19 @@ swap_ops_for_binary_stmt (vec ops, } } +/* Determine if stmt A is in th next list of stmt B. */ +static inline bool +next_stmt_of (gimple a, gimp

[PATCH] Fix PR58775

2013-10-18 Thread Zhenqiang Chen
Hi, The patch fixes PR58775. Bootstrap and no make check regression on X86-64? Is it OK? Thanks! -Zhenqiang ChangeLog: 2013-10-18 Zhenqiang Chen PR tree-optimization/58775 * tree-ssa-reassoc.c (next_stmt_of): New function. (not_dominated_by): Check next_stmt_of for