Re: [PATCH PR68906]

2015-12-17 Thread Richard Biener
On Thu, Dec 17, 2015 at 10:21 AM, Yuri Rumyantsev wrote: > Richard, > > Here is modified patch which checks only that exit block belongs to loop. > > Bootstrapping and regression testing were successful. > Is it OK for trunk? Ok with the spurious @@ -441,7 +441,7 @@ iterations = estimated_loo

Re: [PATCH PR68906]

2015-12-17 Thread Yuri Rumyantsev
Richard, Here is modified patch which checks only that exit block belongs to loop. Bootstrapping and regression testing were successful. Is it OK for trunk? ChangeLog: 2014-12-17 Yuri Rumyantsev PR tree-optimization/68906 * tree-ssa-loop-unswitch.c (tree_unswitch_outer_loop): Add a check th

Re: [PATCH PR68906]

2015-12-16 Thread Richard Biener
On Wed, Dec 16, 2015 at 3:36 PM, Yuri Rumyantsev wrote: > Richard, > > Here is updated patch which includes (1) a test on exit proposed by > you and (2) another test from PR68021 which is caught by new check on > counted loop. Outer-loop unswitching is not performed for both new > tests. As said

Re: [PATCH PR68906]

2015-12-16 Thread Yuri Rumyantsev
Richard, Here is updated patch which includes (1) a test on exit proposed by you and (2) another test from PR68021 which is caught by new check on counted loop. Outer-loop unswitching is not performed for both new tests. Bootstrapping and regression testing did not show any new failures. Is it O

Re: [PATCH PR68906]

2015-12-16 Thread Richard Biener
On Wed, Dec 16, 2015 at 1:14 PM, Yuri Rumyantsev wrote: > Hi All, > > Here is simple patch which cures the issue with outer-loop unswitching > - added invocation of number_of_latch_executions() to reject > unswitching for non-iterated loops. > > Bootstrapping and regression testing did not show an

[PATCH PR68906]

2015-12-16 Thread Yuri Rumyantsev
Hi All, Here is simple patch which cures the issue with outer-loop unswitching - added invocation of number_of_latch_executions() to reject unswitching for non-iterated loops. Bootstrapping and regression testing did not show any new failures. Is it OK for trunk? ChangeLog: 2014-12-16 Yuri Rum