Re: [PATCH] Compact SSA version namespace when releasing the freelist

2012-04-07 Thread Diego Novillo
On 4/6/12 1:48 PM, Richard Guenther wrote: That's with changed statistics accounting, it's a histogram for the per-function number SSA names that got released, thus for 16 functions we released zero SSA names, for 13 functions one. Great, thanks. Is this documented in the internals documentat

Re: [PATCH] Compact SSA version namespace when releasing the freelist

2012-04-06 Thread Richard Guenther
On Thu, Apr 5, 2012 at 5:27 PM, Diego Novillo wrote: > On 4/5/12 6:13 AM, Richard Guenther wrote: >> >> >> Currently we release the memory for released SSA names (which we keep >> around on a freelist, mainly to try to keep the SSA version namespace >> dense) after early optimizations.  This has t

Re: [PATCH] Compact SSA version namespace when releasing the freelist

2012-04-05 Thread Diego Novillo
On 4/5/12 6:13 AM, Richard Guenther wrote: Currently we release the memory for released SSA names (which we keep around on a freelist, mainly to try to keep the SSA version namespace dense) after early optimizations. This has the negative impact (as documented) of permanently establishing holes