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: [PATCH] PR ld/20828: Fix linker script symbols wrongly forced local with section GC


On Mon, Jan 16, 2017 at 10:10:57PM +0000, Maciej W. Rozycki wrote:
> 	bfd/
> 	PR ld/20828
> 	* elflink.c (bfd_elf_record_link_assignment): Clear any 
> 	`forced_local' marking for DSO symbols that are not being 
> 	provided.
> 
> 	ld/
> 	PR ld/20828
> 	* testsuite/ld-elf/pr20828-1.sd: New test.
> 	* testsuite/ld-elf/pr20828-2a.sd: New test.
> 	* testsuite/ld-elf/pr20828-2b.sd: New test.
> 	* testsuite/ld-elf/pr20828.ld: New test linker script.
> 	* testsuite/ld-elf/pr20828.ver: New test version script.
> 	* testsuite/ld-elf/pr20828.s: New test source.
> 	* testsuite/ld-elf/shared.exp: Run the new test.

OK.  Should go on the branch too.

>  I do hope the patch description clearly indicates the bug conditions;

Excellently.

-- 
Alan Modra
Australia Development Lab, IBM


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