From: H.J. Lu Date: Fri, 16 Oct 2015 11:17:49 +0000 (-0700) Subject: Check if symbol is defined when converting mov to lea X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=75a06c790f96f07e859be95d3c3c084a718afeef;p=binutils-gdb.git Check if symbol is defined when converting mov to lea We need to check if symbol is defined when converting mov to lea since SYMBOL_REFERENCES_LOCAL may return true on hidden undefined symbols. * elf32-i386.c (elf_i386_convert_mov_to_lea): Check if symbol is defined. * elf64-x86-64.c (elf_x86_64_convert_mov_to_lea): Likewise. --- diff --git a/bfd/ChangeLog b/bfd/ChangeLog index d26a526d424..d3715949f36 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,9 @@ +2015-10-16 H.J. Lu + + * elf32-i386.c (elf_i386_convert_mov_to_lea): Check if symbol + is defined. + * elf64-x86-64.c (elf_x86_64_convert_mov_to_lea): Likewise. + 2015-10-16 H.J. Lu * elf32-i386.c (elf_i386_convert_mov_to_lea): Don't check diff --git a/bfd/elf32-i386.c b/bfd/elf32-i386.c index 2e5c04da634..8c1dadb4cf3 100644 --- a/bfd/elf32-i386.c +++ b/bfd/elf32-i386.c @@ -2815,7 +2815,9 @@ elf_i386_convert_mov_to_lea (bfd *abfd, asection *sec, /* STT_GNU_IFUNC must keep R_386_GOT32 relocation. We also avoid optimizing _DYNAMIC since ld.so may use its link-time address. */ - if (h->type != STT_GNU_IFUNC + if ((h->root.type == bfd_link_hash_defined + || h->root.type == bfd_link_hash_defweak) + && h->type != STT_GNU_IFUNC && h != htab->elf.hdynamic && SYMBOL_REFERENCES_LOCAL (link_info, h) && irel->r_offset >= 2 diff --git a/bfd/elf64-x86-64.c b/bfd/elf64-x86-64.c index aad324490c9..dc5fd0db430 100644 --- a/bfd/elf64-x86-64.c +++ b/bfd/elf64-x86-64.c @@ -3074,7 +3074,9 @@ elf_x86_64_convert_mov_to_lea (bfd *abfd, asection *sec, /* STT_GNU_IFUNC must keep R_X86_64_GOTPCREL relocation. We also avoid optimizing _DYNAMIC since ld.so may use its link-time address. */ - if (h->type != STT_GNU_IFUNC + if ((h->root.type == bfd_link_hash_defined + || h->root.type == bfd_link_hash_defweak) + && h->type != STT_GNU_IFUNC && h != htab->elf.hdynamic && SYMBOL_REFERENCES_LOCAL (link_info, h)) {