> 
> Why is VEC any better in terms of density ? Are you suggesting using a
> hash table?
It is not any better, but we usually use VEC for variably sized arrays like
this one.  Not that I would be big fan of its API, but at least it fives
bounds checking that would catch bugs like one you are fixing more reliably.

Honza
> 
> 
> , making init_pid_map to allocate it cleared and add also
> > test that VEC_index (pid_map, pid) is not NULL in addition to the bound 
> > check
> > above.
> >
> > Also we ought to free pids once the IPA profiling pass is done. I.e. at the 
> > end of tree_profling
> > function.
> 
> Yes.
> 
> Thanks,
> 
> David
> 
> >
> > Honza
> >

Reply via email to