This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
Re: PATCH: Speed up ELF section merge
On Fri, May 06, 2005 at 03:38:57PM +0100, Dave Korn wrote:
> ----Original Message----
> >From: H. J. Lu
> >Sent: 02 May 2005 03:29
>
> > Here is the new one.
> >
> >
> > H.J.
> > ----
> > 2005-05-01 H.J. Lu <hongjiu.lu@intel.com>
> >
> > * merge.c (sec_merge_init): Call bfd_hash_table_init_n with
> > hash table size 16699 instead of bfd_hash_table_init.
> >
> > --- bfd/merge.c.hash 2005-04-15 18:51:49.000000000 -0700
> > +++ bfd/merge.c 2005-05-01 19:25:42.000000000 -0700
> > @@ -241,7 +241,8 @@ sec_merge_init (unsigned int entsize, bf
> > if (table == NULL)
> > return NULL;
> >
> > - if (! bfd_hash_table_init (&table->table, sec_merge_hash_newfunc))
> > + if (! bfd_hash_table_init_n (&table->table, sec_merge_hash_newfunc,
> > + 16699))
> > {
> > free (table);
> > return NULL;
>
>
>
> H.J., IIUIC this patch can't do anything to support the ld command line
> options --hash-size=<NUMBER> and --reduce-memory-overheads; wouldn't it be
> worthwhile providing a means of controlling whether it uses a large hash
> table or not?
>
I am full on my plate. A patch is more than welcome. Thanks.
H.J.