Re: [PATCH] Fix PR81090, properly free niter estimates

2017-06-21 Thread Christophe Lyon
On 20 June 2017 at 11:45, Richard Biener wrote: > On Tue, 20 Jun 2017, Alan Hayward wrote: > >> >> > On 19 Jun 2017, at 13:35, Richard Biener wrote: >> > >> > On Mon, 19 Jun 2017, Christophe Lyon wrote: >> > >> >> Hi Richard, >> >> >> >> On 16 June 2017 at 14:18, Richard Biener wrote: >> >>> On

Re: [PATCH] Fix PR81090, properly free niter estimates

2017-06-20 Thread Richard Biener
On Tue, 20 Jun 2017, Alan Hayward wrote: > > > On 19 Jun 2017, at 13:35, Richard Biener wrote: > > > > On Mon, 19 Jun 2017, Christophe Lyon wrote: > > > >> Hi Richard, > >> > >> On 16 June 2017 at 14:18, Richard Biener wrote: > >>> On Wed, 14 Jun 2017, Richard Biener wrote: > >>> > >

Re: [PATCH] Fix PR81090, properly free niter estimates

2017-06-20 Thread Alan Hayward
> On 19 Jun 2017, at 13:35, Richard Biener wrote: > > On Mon, 19 Jun 2017, Christophe Lyon wrote: > >> Hi Richard, >> >> On 16 June 2017 at 14:18, Richard Biener wrote: >>> On Wed, 14 Jun 2017, Richard Biener wrote: >>> niter estimates are not kept up-to-date (they reference gimpl

Re: [PATCH] Fix PR81090, properly free niter estimates

2017-06-19 Thread Richard Biener
On Mon, 19 Jun 2017, Christophe Lyon wrote: > Hi Richard, > > On 16 June 2017 at 14:18, Richard Biener wrote: > > On Wed, 14 Jun 2017, Richard Biener wrote: > > > >> > >> niter estimates are not kept up-to-date (they reference gimple stmts > >> and trees) in the keep-loop-stuff infrastructure so

Re: [PATCH] Fix PR81090, properly free niter estimates

2017-06-19 Thread Christophe Lyon
Hi Richard, On 16 June 2017 at 14:18, Richard Biener wrote: > On Wed, 14 Jun 2017, Richard Biener wrote: > >> >> niter estimates are not kept up-to-date (they reference gimple stmts >> and trees) in the keep-loop-stuff infrastructure so similar to the >> SCEV cache we rely on people freeing it af

Re: [PATCH] Fix PR81090, properly free niter estimates

2017-06-16 Thread Richard Biener
On Wed, 14 Jun 2017, Richard Biener wrote: > > niter estimates are not kept up-to-date (they reference gimple stmts > and trees) in the keep-loop-stuff infrastructure so similar to the > SCEV cache we rely on people freeing it after passes. > > The following brings us a step closer to that by fr

[PATCH] Fix PR81090, properly free niter estimates

2017-06-14 Thread Richard Biener
niter estimates are not kept up-to-date (they reference gimple stmts and trees) in the keep-loop-stuff infrastructure so similar to the SCEV cache we rely on people freeing it after passes. The following brings us a step closer to that by freeing them whenever SCEV is invalidated (we only compute