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/19579/21306 Properly turn common symbol into definition


On Thu, Apr 06, 2017 at 10:45:32AM -0700, H.J. Lu wrote:
> Since common symbols that are turned into definitions don't have the
> DEF_REGULAR flag set, we need to check ELF_COMMON_DEF_P for common
> symbols.
> 
> bfd/
> 
> 	PR ld/19579
> 	PR ld/21306
> 	* elf32-s390.c (elf_s390_finish_dynamic_symbol): Check
> 	ELF_COMMON_DEF_P for common symbols.
> 	* elf64-s390.c (elf_s390_finish_dynamic_symbol): Likewise.
> 	* elf64-x86-64.c (elf_x86_64_relocate_section): Likewise.
> 	* elflink.c (_bfd_elf_merge_symbol): Revert commits
> 	202ac193bbbecc96a4978d1ac3d17148253f9b01 and
> 	07492f668d2173da7a2bda3707ff0985e0f460b6.
> 
> ld/
> 
> 	PR ld/19579
> 	PR ld/21306
> 	* testsuite/ld-elf/pr19579a.c (main): Updated.

This is OK.  Thanks for looking at s390 too.  Which other targets do
you have cross-compilers installed or test natively in order to see
19579 failures?  (I'm assuming that's why you made the s390 changes,
and would like to know the targets that might yet need attention.)

-- 
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]