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: RFC: Adding an extra field to Elf_Internal_Sym


On Fri, Nov 26, 2010 at 11:44:27AM +0000, Richard Sandiford wrote:
> One option is to add an internal-only st_type.  There is one free:
> STT_LOPROC + 1.  But we'd then be scuppered if a real ARM-specific
> st_type was defined in future.

You also have STT_LOOS+1 and STT_LOOS+2.  That gives you three to
choose from, and since this is internal only you can change between
those values at will.  I think that's the way I'd go.

> 	* elf-bfd.h (elf_link_hash_entry): Add target_internal.

I'd rather see the flags shrink by two..  Other than that, I really
don't have any objection to your patch.

-- 
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]