This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: New gold plugin interface for getting --wrap symbols
- From: "Sriraman Tallam via binutils" <binutils at sourceware dot org>
- To: Cary Coutant <ccoutant at gmail dot com>
- Cc: binutils <binutils at sourceware dot org>, Xinliang David Li <davidxl at google dot com>, Teresa Johnson <tejohnson at google dot com>
- Date: Thu, 22 Feb 2018 13:59:14 -0800
- Subject: Re: New gold plugin interface for getting --wrap symbols
- Authentication-results: sourceware.org; auth=none
- References: <CAAs8Hmzq+11BitwNknrA8o6GNPXiWK+7asWU8rBaFRcsMs16_Q@mail.gmail.com> <CAAs8HmxjaR087175sZRAa19VUz6xca4j+oE8riH2uf0qRhStUA@mail.gmail.com> <CAJimCsHX_iis4cAMku73t=3n90wAhz+4mz4-FFd6+bA=qmMTqQ@mail.gmail.com> <CAAs8Hmx4608QS8wEToFMrL6oLcOiHa4COEEq_bSzNxa+FuVa8w@mail.gmail.com> <CAJimCsFXzdEFQ_B=9n73jXXfd_Hwb0Rn=xu0n1y5fCvwZ-idug@mail.gmail.com>
- Reply-to: Sriraman Tallam <tmsriram at google dot com>
This patch is now committed and pushed to binutils and I will push
plugin-api.h changes to GCC once Stage1 opens again.
On Fri, Dec 1, 2017 at 4:32 PM, Cary Coutant <ccoutant@gmail.com> wrote:
>> 2017-11-21 Sriraman Tallam <tmsriram@google.com>
>>
>> * plugin.cc (get_wrap_symbols): New plugin interface.
>> (load): Add get_wrap_symbols to transfer vector.
>> * plugin-api.h (ld_plugin_get_wrap_symbols): New plugin interface.
>> * options.h (options::String_set::size_type): New macro function.
>
> Change this to: (DEFINE_string): Add varname_size() method.
>
>> * testsuite/plugin_test.c (onload): Call and check get_wrap_symbols
>> interface.
>> * testsuite/plugin_test_wrap_symbols.sh: New test script.
>> * testsuite/plugin_test_wrap_symbols_1.cc: New file.
>> * testsuite/plugin_test_wrap_symbols_2.cc: New file.
>> * testsuite/Makefile.am (plugin_test_wrap_symbols): New test.
>> * testsuite/Makefile.in: Regenerate.
>
> plugin-api.h is in include/ which has its own ChangeLog. That patch
> should be applied first and synced with gcc (not sure if they'll take
> this during Stage 3, but I think it's harmless enough to be OK). The
> gold part is OK after that's checked in.
>
> -cary