106 GCC HEAD regressions, 14 new, with your patch on 2004-05-20T16:12:37Z.

Zack Weinberg zack@codesourcery.com
Fri May 21 22:48:00 GMT 2004


Geoff Keating <geoffk@geoffk.org> writes:

> +       * cgraph.c (hash_node, eq_node, cgraph_node, cgraph_remove_node)
> +       (cgraph_varpool_hash_node, eq_cgraph_varpool_node)
> +       (cgraph_varpool_node): Hash on the pointer to the decl, not
> +       the DECL_UID.  Fixes 64-bit bootstrap failure.
>
> As far as I can tell, this patch is just plain wrong.  You can't make
> hashes based on pointers that are saved by PCH unless you either (a)
> arrange for them to be reordered before writing the PCH, or (b) don't
> mind getting a 'not found' result when the data is actually in the
> hash table.

Yes, I realize this now.  I am trying to come up with a new patch, but
my amd64 machine is not cooperating.

zw



More information about the Gcc-regression mailing list