[PATCH] Add plugin interface to LD [0/4] Support ELF symbol visibility

[Dave Korn]

On 23/09/2010 17:18, Richard Henderson wrote:
> On 09/22/2010 10:29 PM, Dave Korn wrote:
>> - ELF visibility handling doesn't need to be a first priority because ELF
>> targets should be using GOLD for best results anyway; this patch is sufficient
>> for non-ELF targets that don't have the option of using GOLD and don't need
>> visibility support anyway.
> 
> "Should be", but there's a whole collection of elf targets that
> are unlikely to ever be re-written for GOLD.
> 
> But you're right that it needn't be a first priority.
> 
> 
> r~


  Yeah, but it's like about /four/ extra statements or so... so here it is!

  ld/ChangeLog:

	* plugin.c (asymbol_from_plugin_symbol): If the bfd is an ELF bfd,
	find the elf symbol data and set the visibility in the st_other field.

  ld/testsuite/ChangeLog:

	* plugin-ignore.d: New dump test control script.
	* plugin-vis-1.d: Likewise.
	* plugin.exp: Add list of ELF-only tests and run them if testing on
	an ELF target.

    cheers,
      DaveK

-------------- next part --------------
A non-text attachment was scrubbed...
Name: ld-plugin-api-5-elf-vis.diff
Type: text/x-c
Size: 3691 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20100924/2d567407/attachment.bin>