Re: [PATCH] Fix compute_bucket_count

[Jakub Jelinek <jakub at redhat dot com>]

On Tue, Jul 04, 2006 at 12:01:19PM +0200, Jakub Jelinek wrote:
> I have some raw data for the same binaries/libraries built without my first
> DT_GNU_HASH patch and after it (that patch included also the arch_size / 8
> -> bed->size_of_hash_entry change, nsyms -> dynsymcount in ld -O1 code
> and dynsymcount -> (hashcodesp - hashcodes) in non-optimizing compute_bucket_count).
> For .gnu.hash, one shouldn't look at the st_size of .gnu.hash section,
> that was decreased by (2 * nchains - 1) * 4 with the patch from yesterday.

Well, don't look at the histograms for .gnu.hash in that data either, just
for .hash.  I just realized I used readelf -I from binutils build with the
latest DT_GNU_HASH patch (i.e. the new, smaller .gnu.hash format) on the
binaries/libraries created by older binutils (the older, bigger .gnu.hash
format).  I was shocked to see e.g. chains with length 48 in there only
to figure later that they aren't that long.  If you are interested, will
repost new ppc.after.bz2 and ia64.after.bz2 data, created with the matching
readelf.  Sorry.

	Jakub