RE: Move cgraph_node_set and varpool_node_set out of GGC and make them use pointer_map

2011-05-04 Thread Ian Bolton
> Hi, > I always considered the cgrpah_node_set/varpool_node_set to be > overengineered > but they also turned out to be quite ineffective since we do quite a > lot of > queries into them during stremaing out. > > This patch moves them to pointer_map, like I did for streamer cache. > While > doing

Re: Move cgraph_node_set and varpool_node_set out of GGC and make them use pointer_map

2011-05-03 Thread Jan Hubicka
> > We have both a vector and a pointer-map. Why not simply use a > pointer-map only?! I see this may need more re-structuring, eventually Well, pointer-maps would be randomly ordered (sensitive to pointer values) and thus we would give different .o files depending on memory layout. But yes, co

Re: Move cgraph_node_set and varpool_node_set out of GGC and make them use pointer_map

2011-05-03 Thread Richard Guenther
On Tue, 3 May 2011, Jan Hubicka wrote: > Hi, > I always considered the cgrpah_node_set/varpool_node_set to be overengineered > but they also turned out to be quite ineffective since we do quite a lot of > queries into them during stremaing out. > > This patch moves them to pointer_map, like I did