This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] S/390: STT_GNU_IFUNC support


On Mon, Jul 02, 2012 at 12:00:15PM +0200, Andreas Krebbel wrote:
> Glibc and Binutils testsuites are clean with that patch.

To get that far, your code must be reasonable.  :-)
I noticed a couple of things are missing.

- You should be setting ELFOSABI_GNU on detecting ifunc symbols.  This
  is normally arranged by setting has_gnu_symbols in add_symbol_hook.
- sweep_hook code handling local ifunc.

-- 
Alan Modra
Australia Development Lab, IBM


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]