This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] Add ARM relocation constants to elf/elf.h
- From: Petr Machata <pmachata at redhat dot com>
- To: Roland McGrath <roland at hack dot frob dot com>
- Cc: "Joseph S. Myers" <joseph at codesourcery dot com>
- Cc: libc-alpha at sourceware dot org
- Date: Wed, 23 Jan 2013 13:02:29 +0100
- Subject: Re: [PATCH] Add ARM relocation constants to elf/elf.h
- References: <m2ehhdyypc.fsf@redhat.com><20130123004615.A057D2C08F@topped-with-meat.com>
Roland McGrath <roland@hack.frob.com> writes:
> We strongly prefer to have a short comment on every reloc type #define.
I don't think I have enough expertise to concisely explain what every
relocation does. I can at least put in e.g. what instructions each
static relocation applies to, and add snippets from the standard where
available. Joseph, what is your preference? FWIW, few relocation
values in ARM block have any comments at all.
> The R_ARM_PRIVATE_* ones look a bit suspect to me. That spec document does
> describe that space of names, though it doesn't list them individually.
> But I really wonder if anybody actually wants or uses those names.
> binutils doesn't use them.
Right, makes no sense to export the names. The numbers are guaranteed
not to be allocated, but hardly anyone will call them this.
> But those are just my immediate reactions. The specifics of what should or
> shouldn't go in are up to Joseph as the ARM maintainer.
Whom I have now added to the loop.
Thanks,
PM