Re: plugin-api.h patch to add a new interface for linker plugins

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: plugin-api.h patch to add a new interface for linker plugins

Sourceware - binutils list mailing list
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 <[hidden email]> 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 <[hidden email]> 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

plugin_api_patch.txt (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: plugin-api.h patch to add a new interface for linker plugins

Sourceware - binutils list mailing list
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 <[hidden email]> 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 <[hidden email]> 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 <[hidden email]> 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

plugin_api_patch.txt (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: plugin-api.h patch to add a new interface for linker plugins

Cary Coutant-3
> 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
Reply | Threaded
Open this post in threaded view
|

Re: plugin-api.h patch to add a new interface for linker plugins

Nick Clifton
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


Reply | Threaded
Open this post in threaded view
|

Re: plugin-api.h patch to add a new interface for linker plugins

Jeff Law
In reply to this post by Sourceware - binutils list mailing list
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 <[hidden email]> 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 <[hidden email]> 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 <[hidden email]> 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