Re: [PATCH 4/5] remove param$N_is usage

2014-11-13 Thread Trevor Saunders
On Thu, Nov 13, 2014 at 05:49:47PM -0700, Jeff Law wrote: > On 11/12/14 22:55, tsaund...@mozilla.com wrote: > >From: Trevor Saunders > > > >Hi, > > > >The only user of this is splay_tree. We only have a couple splay trees in > >ggc > >memory, and it wasn't clear to me any of them were tree based

Re: [PATCH 4/5] remove param$N_is usage

2014-11-13 Thread Jeff Law
On 11/12/14 22:55, tsaund...@mozilla.com wrote: From: Trevor Saunders Hi, The only user of this is splay_tree. We only have a couple splay trees in ggc memory, and it wasn't clear to me any of them were tree based instead of hash based for performance reasons, so I chose to just convert them

[PATCH 4/5] remove param$N_is usage

2014-11-12 Thread tsaunders
From: Trevor Saunders Hi, The only user of this is splay_tree. We only have a couple splay trees in ggc memory, and it wasn't clear to me any of them were tree based instead of hash based for performance reasons, so I chose to just convert them to hash_map rather than writing a templated splay