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: [committed, PATCH] x86: Add POINTER_LOCAL_IFUNC_P/PLT_LOCAL_IFUNC_P


On 10/6/17, Pedro Alves <palves@redhat.com> wrote:
> On 10/06/2017 08:38 AM, H.J. Lu wrote:
>> +/* TRUE if this is a PLT reference to a local IFUNC.  */
>> +#define PLT_LOCAL_IFUNC_P(INFO, H) \
>> +  ((H)->dynindx == -1 \
>> +   || ((bfd_link_executable (INFO) \
>> +	|| ELF_ST_VISIBILITY ((H)->other) != STV_DEFAULT) \
>> +	&& (H)->def_regular \
>> +	&& (H)->type == STT_GNU_IFUNC))
>> +
>
> OOC, is there a good reason these things are macros
> instead of (static inline) functions?
>
>

Just a habit.   Care to submit a patch?

-- 
H.J.


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