Re: [PATCH] Fix PR77514

2016-09-18 Thread Richard Biener
On Fri, 16 Sep 2016, Richard Biener wrote: > On Fri, 16 Sep 2016, Bernhard Reutner-Fischer wrote: > > > On 15 September 2016 09:08:36 CEST, Richard Biener > > wrote: > > > > > >Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. > > > > > > > >+ /* Likewise if we simplified

Re: [PATCH] Fix PR77514

2016-09-16 Thread Michael Matz
Hi, On Fri, 16 Sep 2016, Richard Biener wrote: > We don't -- I guess it would be an interesting transform but difficult > in this case as gimple_get_lhs might have side-effects we cannot skip. If there are side-effects it's anyway not going to be very profitable. loop-splitting is mostly an e

Re: [PATCH] Fix PR77514

2016-09-16 Thread Richard Biener
On Fri, 16 Sep 2016, Bernhard Reutner-Fischer wrote: > On 15 September 2016 09:08:36 CEST, Richard Biener wrote: > > > >Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. > > > > >+ /* Likewise if we simplified to sth not queued for insertion. */ > >+ bool found = false; >

Re: [PATCH] Fix PR77514

2016-09-16 Thread Bernhard Reutner-Fischer
On 15 September 2016 09:08:36 CEST, Richard Biener wrote: > >Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. > >+ /* Likewise if we simplified to sth not queued for insertion. */ >+ bool found = false; >+ gsi = gsi_start (forced_stmts); >+ for (; !gsi_end_p (gsi); gsi_

[PATCH] Fix PR77514

2016-09-15 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2016-09-15 Richard Biener PR tree-optimization/77514 * tree-ssa-pre.c (create_expression_by_pieces): Handle garbage only forced_stmts sequence. * gcc.dg/torture/pr77514.c: New tes