On Mon, Apr 4, 2011 at 8:49 PM, Nathan Froyd <froy...@codesourcery.com> wrote: > This patch does just what $SUBJECT suggests. Benefits: > > - Smaller data structures in combine; > - Freeing LOG_LINKS becomes much easier (don't have to transfer > everything to the INSN_LIST free list); > > Potential downsides: > > - Less sharing of INSN_LIST nodes might mean more cache thrashing. > > Bootstrapped on x86_64-unknown-linux-gnu. WDYT...OK to commit?
It looks like LOG_LINKs are allocated once. An alloc pool is interesting if you allocate and free objects of the same size all the time. In this case, I'd say an obstack would be a simpler and better choice. I know: Bikeshed, etc. Ciao! Steven
