hppa: Check UNDEFWEAK_NO_DYNAMIC_RELOC
Don't generate dynamic relocation against weak undefined symbol if it
is resolved to zero. FIXME: UNDEFWEAK_NO_DYNAMIC_RELOC may need to be
checked in more places.
PR ld/22269
* elf32-hppa.c (ensure_undef_dynamic): Don't make undefined
symbol dynamic if UNDEFWEAK_NO_DYNAMIC_RELOC is true.
(allocate_dynrelocs): Discard dynamic relocations if
UNDEFWEAK_NO_DYNAMIC_RELOC is true.
(elf32_hppa_relocate_section): Don't generate dynamic
relocation if UNDEFWEAK_NO_DYNAMIC_RELOC is true.
diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 3011aa2..34dde75 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,6 +1,16 @@
2017-10-14 H.J. Lu <hongjiu.lu@intel.com>
PR ld/22269
+ * elf32-hppa.c (ensure_undef_dynamic): Don't make undefined
+ symbol dynamic if UNDEFWEAK_NO_DYNAMIC_RELOC is true.
+ (allocate_dynrelocs): Discard dynamic relocations if
+ UNDEFWEAK_NO_DYNAMIC_RELOC is true.
+ (elf32_hppa_relocate_section): Don't generate dynamic
+ relocation if UNDEFWEAK_NO_DYNAMIC_RELOC is true.
+
+2017-10-14 H.J. Lu <hongjiu.lu@intel.com>
+
+ PR ld/22269
* elfnn-ia64.c (elfNN_ia64_check_relocs): Don't allocate
dynamic relocation if UNDEFWEAK_NO_DYNAMIC_RELOC is true.
(allocate_fptr): Don't allocate function pointer if
diff --git a/bfd/elf32-hppa.c b/bfd/elf32-hppa.c
index 3adac6d..9a195be 100644
--- a/bfd/elf32-hppa.c
+++ b/bfd/elf32-hppa.c
@@ -1933,6 +1933,7 @@
&& eh->dynindx == -1
&& !eh->forced_local
&& eh->type != STT_PARISC_MILLI
+ && !UNDEFWEAK_NO_DYNAMIC_RELOC (info, eh)
&& ELF_ST_VISIBILITY (eh->other) == STV_DEFAULT)
return bfd_elf_link_record_dynamic_symbol (info, eh);
return TRUE;
@@ -2077,7 +2078,8 @@
/* Discard relocs on undefined syms with non-default visibility. */
if ((eh->root.type == bfd_link_hash_undefined
|| eh->root.type == bfd_link_hash_undefweak)
- && ELF_ST_VISIBILITY (eh->other) != STV_DEFAULT)
+ && (ELF_ST_VISIBILITY (eh->other) != STV_DEFAULT
+ || UNDEFWEAK_NO_DYNAMIC_RELOC (info, eh)))
hh->dyn_relocs = NULL;
#if RELATIVE_DYNRELOCS
@@ -3672,6 +3674,7 @@
const char *sym_name;
bfd_boolean plabel;
bfd_boolean warned_undef;
+ bfd_boolean resolved_to_zero;
r_type = ELF32_R_TYPE (rela->r_info);
if (r_type >= (unsigned int) R_PARISC_UNIMPLEMENTED)
@@ -3735,6 +3738,9 @@
if (bfd_link_relocatable (info))
continue;
+ resolved_to_zero = (hh != NULL
+ && UNDEFWEAK_NO_DYNAMIC_RELOC (info, &hh->eh));
+
/* Do any required modifications to the relocation value, and
determine what types of dynamic info we need to output, if
any. */
@@ -3965,7 +3971,8 @@
there all files have not been loaded. */
if ((bfd_link_pic (info)
&& (hh == NULL
- || ELF_ST_VISIBILITY (hh->eh.other) == STV_DEFAULT
+ || (ELF_ST_VISIBILITY (hh->eh.other) == STV_DEFAULT
+ && !resolved_to_zero)
|| hh->eh.root.type != bfd_link_hash_undefweak)
&& (IS_ABSOLUTE_RELOC (r_type)
|| !SYMBOL_CALLS_LOCAL (info, &hh->eh)))
