From mboxrd@z Thu Jan 1 00:00:00 1970 From: hjl@lucon.org (H.J. Lu) To: rth@cygnus.com Cc: manfred@s-direktnet.de, bfd@cygnus.com, gas2@cygnus.com, Manfred.Hollstein@ks.sel.alcatel.de Subject: Re: Bugs in recent snapshots on Solaris Date: Fri, 20 Mar 1998 08:19:00 -0000 Message-id: References: <19980320023241.39647@dot.cygnus.com> X-SW-Source: 1998/msg00022.html > > > - /* If this symbol is not defined in a regular file, and we are > > - not generating a shared library, then set the symbol to this > > - location in the .plt. This is required to make function > > - pointers compare as equal between the normal executable and > > - the shared library. */ > > - if (! info->shared > > - && (h->elf_link_hash_flags & ELF_LINK_HASH_DEF_REGULAR) == 0) > > - { > > - h->root.u.def.section = s; > > - h->root.u.def.value = s->_raw_size; > > - } > > + /* Set the symbol to this location in the .plt. This is > > + required to make function pointers compare as equal between > > + the normal executable and the shared library. */ > > + h->root.u.def.section = s; > > + h->root.u.def.value = s->_raw_size; > > This was the offending bit. The other part is righteous. > Somewhere I got a bit overeager. > Is that fixed now in the snapshot? I need to make a new binutils for Linux to get this and an x86 bug fixed. Thanks. -- H.J. Lu (hjl@gnu.org)