* plugin-api.h patch to add a new interface for linker plugins @ 2017-12-08 19:01 Sriraman Tallam via gcc-patches 2017-12-08 19:02 ` Sriraman Tallam via gcc-patches 0 siblings, 1 reply; 7+ messages in thread From: Sriraman Tallam via gcc-patches @ 2017-12-08 19:01 UTC (permalink / raw) To: GCC Patches; +Cc: Cary Coutant, Teresa Johnson Hi, This patch was approved for binutils by Cary: https://sourceware.org/ml/binutils/2017-12/msg00023.html Is it ok to apply this to GCC include/plugin-api.h ? Thanks Sri ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: plugin-api.h patch to add a new interface for linker plugins 2017-12-08 19:01 plugin-api.h patch to add a new interface for linker plugins Sriraman Tallam via gcc-patches @ 2017-12-08 19:02 ` Sriraman Tallam via gcc-patches 2018-02-15 21:52 ` Sriraman Tallam via gcc-patches 0 siblings, 1 reply; 7+ messages in thread From: Sriraman Tallam via gcc-patches @ 2017-12-08 19:02 UTC (permalink / raw) To: GCC Patches; +Cc: Cary Coutant, Teresa Johnson [-- Attachment #1: Type: text/plain, Size: 360 bytes --] Patch attached. * plugin-api.h (ld_plugin_get_wrap_symbols): New plugin interface. On Fri, Dec 8, 2017 at 11:01 AM, Sriraman Tallam <tmsriram@google.com> wrote: > Hi, > > This patch was approved for binutils by Cary: > https://sourceware.org/ml/binutils/2017-12/msg00023.html > > Is it ok to apply this to GCC include/plugin-api.h ? > > Thanks > Sri [-- Attachment #2: plugin_api_patch.txt --] [-- Type: text/plain, Size: 1335 bytes --] * plugin-api.h (ld_plugin_get_wrap_symbols): New plugin interface. Index: include/plugin-api.h =================================================================== --- include/plugin-api.h (revision 255515) +++ include/plugin-api.h (working copy) @@ -378,7 +378,15 @@ enum ld_plugin_status (*ld_plugin_register_new_input) (ld_plugin_new_input_handler handler); +/* The linker's interface for getting the list of wrapped symbols using the + --wrap option. This sets *NUM_SYMBOLS to number of wrapped symbols and + *WRAP_SYMBOL_LIST to the list of wrapped symbols. */ +typedef +enum ld_plugin_status +(*ld_plugin_get_wrap_symbols) (uint64_t *num_symbols, + const char ***wrap_symbol_list); + enum ld_plugin_level { LDPL_INFO, @@ -422,7 +430,8 @@ LDPT_GET_SYMBOLS_V3 = 28, LDPT_GET_INPUT_SECTION_ALIGNMENT = 29, LDPT_GET_INPUT_SECTION_SIZE = 30, - LDPT_REGISTER_NEW_INPUT_HOOK = 31 + LDPT_REGISTER_NEW_INPUT_HOOK = 31, + LDPT_GET_WRAP_SYMBOLS = 32 }; /* The plugin transfer vector. */ @@ -457,6 +466,7 @@ ld_plugin_get_input_section_alignment tv_get_input_section_alignment; ld_plugin_get_input_section_size tv_get_input_section_size; ld_plugin_register_new_input tv_register_new_input; + ld_plugin_get_wrap_symbols tv_get_wrap_symbols; } tv_u; }; ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: plugin-api.h patch to add a new interface for linker plugins 2017-12-08 19:02 ` Sriraman Tallam via gcc-patches @ 2018-02-15 21:52 ` Sriraman Tallam via gcc-patches 2018-02-20 17:36 ` Sriraman Tallam via gcc-patches 0 siblings, 1 reply; 7+ messages in thread From: Sriraman Tallam via gcc-patches @ 2018-02-15 21:52 UTC (permalink / raw) To: GCC Patches Cc: Cary Coutant, Teresa Johnson, binutils, Xinliang David Li, H.J. Lu [-- Attachment #1: Type: text/plain, Size: 754 bytes --] Ping, this patch was approved for binutils by Cary: https://sourceware.org/ml/binutils/2017-12/msg00023.html Is it ok to apply this to GCC include/plugin-api.h now? It is a pretty small change. Patch attached. * plugin-api.h (ld_plugin_get_wrap_symbols): New plugin interface. Thanks Sri On Fri, Dec 8, 2017 at 11:02 AM, Sriraman Tallam <tmsriram@google.com> wrote: > Patch attached. > > * plugin-api.h (ld_plugin_get_wrap_symbols): New > plugin interface. > > On Fri, Dec 8, 2017 at 11:01 AM, Sriraman Tallam <tmsriram@google.com> wrote: >> Hi, >> >> This patch was approved for binutils by Cary: >> https://sourceware.org/ml/binutils/2017-12/msg00023.html >> >> Is it ok to apply this to GCC include/plugin-api.h ? >> >> Thanks >> Sri [-- Attachment #2: plugin_api_patch.txt --] [-- Type: text/plain, Size: 1335 bytes --] * plugin-api.h (ld_plugin_get_wrap_symbols): New plugin interface. Index: include/plugin-api.h =================================================================== --- include/plugin-api.h (revision 255515) +++ include/plugin-api.h (working copy) @@ -378,7 +378,15 @@ enum ld_plugin_status (*ld_plugin_register_new_input) (ld_plugin_new_input_handler handler); +/* The linker's interface for getting the list of wrapped symbols using the + --wrap option. This sets *NUM_SYMBOLS to number of wrapped symbols and + *WRAP_SYMBOL_LIST to the list of wrapped symbols. */ +typedef +enum ld_plugin_status +(*ld_plugin_get_wrap_symbols) (uint64_t *num_symbols, + const char ***wrap_symbol_list); + enum ld_plugin_level { LDPL_INFO, @@ -422,7 +430,8 @@ LDPT_GET_SYMBOLS_V3 = 28, LDPT_GET_INPUT_SECTION_ALIGNMENT = 29, LDPT_GET_INPUT_SECTION_SIZE = 30, - LDPT_REGISTER_NEW_INPUT_HOOK = 31 + LDPT_REGISTER_NEW_INPUT_HOOK = 31, + LDPT_GET_WRAP_SYMBOLS = 32 }; /* The plugin transfer vector. */ @@ -457,6 +466,7 @@ ld_plugin_get_input_section_alignment tv_get_input_section_alignment; ld_plugin_get_input_section_size tv_get_input_section_size; ld_plugin_register_new_input tv_register_new_input; + ld_plugin_get_wrap_symbols tv_get_wrap_symbols; } tv_u; }; ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: plugin-api.h patch to add a new interface for linker plugins 2018-02-15 21:52 ` Sriraman Tallam via gcc-patches @ 2018-02-20 17:36 ` Sriraman Tallam via gcc-patches 2018-02-20 23:35 ` Cary Coutant 2018-05-01 18:37 ` Jeff Law 0 siblings, 2 replies; 7+ messages in thread From: Sriraman Tallam via gcc-patches @ 2018-02-20 17:36 UTC (permalink / raw) To: GCC Patches Cc: Cary Coutant, Teresa Johnson, binutils, Xinliang David Li, H.J. Lu [-- Attachment #1: Type: text/plain, Size: 1019 bytes --] Ping. Is this alright to apply now or should I wait for Stage 1? * plugin-api.h (ld_plugin_get_wrap_symbols): New plugin interface. Thanks Sri On Thu, Feb 15, 2018 at 1:52 PM, Sriraman Tallam <tmsriram@google.com> wrote: > Ping, this patch was approved for binutils by Cary: > https://sourceware.org/ml/binutils/2017-12/msg00023.html > > Is it ok to apply this to GCC include/plugin-api.h now? It is a > pretty small change. Patch attached. > > * plugin-api.h (ld_plugin_get_wrap_symbols): New > plugin interface. > > > Thanks > Sri > > On Fri, Dec 8, 2017 at 11:02 AM, Sriraman Tallam <tmsriram@google.com> wrote: >> Patch attached. >> >> * plugin-api.h (ld_plugin_get_wrap_symbols): New >> plugin interface. >> >> On Fri, Dec 8, 2017 at 11:01 AM, Sriraman Tallam <tmsriram@google.com> wrote: >>> Hi, >>> >>> This patch was approved for binutils by Cary: >>> https://sourceware.org/ml/binutils/2017-12/msg00023.html >>> >>> Is it ok to apply this to GCC include/plugin-api.h ? >>> >>> Thanks >>> Sri [-- Attachment #2: plugin_api_patch.txt --] [-- Type: text/plain, Size: 1335 bytes --] * plugin-api.h (ld_plugin_get_wrap_symbols): New plugin interface. Index: include/plugin-api.h =================================================================== --- include/plugin-api.h (revision 255515) +++ include/plugin-api.h (working copy) @@ -378,7 +378,15 @@ enum ld_plugin_status (*ld_plugin_register_new_input) (ld_plugin_new_input_handler handler); +/* The linker's interface for getting the list of wrapped symbols using the + --wrap option. This sets *NUM_SYMBOLS to number of wrapped symbols and + *WRAP_SYMBOL_LIST to the list of wrapped symbols. */ +typedef +enum ld_plugin_status +(*ld_plugin_get_wrap_symbols) (uint64_t *num_symbols, + const char ***wrap_symbol_list); + enum ld_plugin_level { LDPL_INFO, @@ -422,7 +430,8 @@ LDPT_GET_SYMBOLS_V3 = 28, LDPT_GET_INPUT_SECTION_ALIGNMENT = 29, LDPT_GET_INPUT_SECTION_SIZE = 30, - LDPT_REGISTER_NEW_INPUT_HOOK = 31 + LDPT_REGISTER_NEW_INPUT_HOOK = 31, + LDPT_GET_WRAP_SYMBOLS = 32 }; /* The plugin transfer vector. */ @@ -457,6 +466,7 @@ ld_plugin_get_input_section_alignment tv_get_input_section_alignment; ld_plugin_get_input_section_size tv_get_input_section_size; ld_plugin_register_new_input tv_register_new_input; + ld_plugin_get_wrap_symbols tv_get_wrap_symbols; } tv_u; }; ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: plugin-api.h patch to add a new interface for linker plugins 2018-02-20 17:36 ` Sriraman Tallam via gcc-patches @ 2018-02-20 23:35 ` Cary Coutant 2018-02-22 11:25 ` Nick Clifton 2018-05-01 18:37 ` Jeff Law 1 sibling, 1 reply; 7+ messages in thread From: Cary Coutant @ 2018-02-20 23:35 UTC (permalink / raw) To: Sriraman Tallam, Nick Clifton Cc: GCC Patches, Teresa Johnson, binutils, Xinliang David Li, H.J. Lu > Ping. Is this alright to apply now or should I wait for Stage 1? > > * plugin-api.h (ld_plugin_get_wrap_symbols): New > plugin interface. I'd say go ahead and apply the patch in binutils, and wait for Stage 1 to sync back to GCC, unless someone there OKs it sooner. Nick, is that OK? -cary ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: plugin-api.h patch to add a new interface for linker plugins 2018-02-20 23:35 ` Cary Coutant @ 2018-02-22 11:25 ` Nick Clifton 0 siblings, 0 replies; 7+ messages in thread From: Nick Clifton @ 2018-02-22 11:25 UTC (permalink / raw) To: Cary Coutant, Sriraman Tallam Cc: GCC Patches, Teresa Johnson, binutils, Xinliang David Li, H.J. Lu Hi Cary, Hi Sriraman, >> Ping. Is this alright to apply now or should I wait for Stage 1? >> >> * plugin-api.h (ld_plugin_get_wrap_symbols): New >> plugin interface. > > I'd say go ahead and apply the patch in binutils, and wait for Stage 1 > to sync back to GCC, unless someone there OKs it sooner. > > Nick, is that OK? Yes, that is fine with me. Cheers Nick ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: plugin-api.h patch to add a new interface for linker plugins 2018-02-20 17:36 ` Sriraman Tallam via gcc-patches 2018-02-20 23:35 ` Cary Coutant @ 2018-05-01 18:37 ` Jeff Law 1 sibling, 0 replies; 7+ messages in thread From: Jeff Law @ 2018-05-01 18:37 UTC (permalink / raw) To: Sriraman Tallam, GCC Patches Cc: Cary Coutant, Teresa Johnson, binutils, Xinliang David Li, H.J. Lu On 02/20/2018 10:35 AM, Sriraman Tallam via binutils wrote: > Ping. Is this alright to apply now or should I wait for Stage 1? > > * plugin-api.h (ld_plugin_get_wrap_symbols): New > plugin interface. > > Thanks > Sri > > On Thu, Feb 15, 2018 at 1:52 PM, Sriraman Tallam <tmsriram@google.com> wrote: >> Ping, this patch was approved for binutils by Cary: >> https://sourceware.org/ml/binutils/2017-12/msg00023.html >> >> Is it ok to apply this to GCC include/plugin-api.h now? It is a >> pretty small change. Patch attached. >> >> * plugin-api.h (ld_plugin_get_wrap_symbols): New >> plugin interface. >> >> >> Thanks >> Sri >> >> On Fri, Dec 8, 2017 at 11:02 AM, Sriraman Tallam <tmsriram@google.com> wrote: >>> Patch attached. >>> >>> * plugin-api.h (ld_plugin_get_wrap_symbols): New >>> plugin interface. >>> >>> On Fri, Dec 8, 2017 at 11:01 AM, Sriraman Tallam <tmsriram@google.com> wrote: >>>> Hi, >>>> >>>> This patch was approved for binutils by Cary: >>>> https://sourceware.org/ml/binutils/2017-12/msg00023.html >>>> >>>> Is it ok to apply this to GCC include/plugin-api.h ? OK for the trunk. jeff ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-05-01 18:37 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-12-08 19:01 plugin-api.h patch to add a new interface for linker plugins Sriraman Tallam via gcc-patches 2017-12-08 19:02 ` Sriraman Tallam via gcc-patches 2018-02-15 21:52 ` Sriraman Tallam via gcc-patches 2018-02-20 17:36 ` Sriraman Tallam via gcc-patches 2018-02-20 23:35 ` Cary Coutant 2018-02-22 11:25 ` Nick Clifton 2018-05-01 18:37 ` Jeff Law
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).