From: Nick Clifton Date: Tue, 11 Jun 2013 07:21:40 +0000 (+0000) Subject: * elf32-rl78.c (rl78_elf_finish_dynamic_sections): Onlly run X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=47fc0f1f211b925f68e4bb8d34b5df5479fd5596;p=binutils-gdb.git * elf32-rl78.c (rl78_elf_finish_dynamic_sections): Onlly run checks if the dynamic sections have been created and relaxation has not been performed. --- diff --git a/bfd/ChangeLog b/bfd/ChangeLog index baff0959443..cfe54f93f62 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,9 @@ +2013-06-11 Nick Clifton + + * elf32-rl78.c (rl78_elf_finish_dynamic_sections): Onlly run + checks if the dynamic sections have been created and relaxation + has not been performed. + 2013-06-07 Will Newton * configure: Regenerate. diff --git a/bfd/elf32-rl78.c b/bfd/elf32-rl78.c index 11d1ecc7457..fb31176da04 100644 --- a/bfd/elf32-rl78.c +++ b/bfd/elf32-rl78.c @@ -1270,24 +1270,28 @@ rl78_elf_finish_dynamic_sections (bfd *abfd ATTRIBUTE_UNUSED, bfd *dynobj; asection *splt; + if (!elf_hash_table (info)->dynamic_sections_created) + return TRUE; + /* As an extra sanity check, verify that all plt entries have been filled in. However, relaxing might have changed the relocs so that some plt entries don't get filled in, so we have to skip this check if we're relaxing. Unfortunately, check_relocs is called before relaxation. */ - if (info->relax_trip > 0) + if (info->relax_trip > 0) + return TRUE; + + if ((dynobj = elf_hash_table (info)->dynobj) != NULL + && (splt = bfd_get_linker_section (dynobj, ".plt")) != NULL) { - if ((dynobj = elf_hash_table (info)->dynobj) != NULL - && (splt = bfd_get_linker_section (dynobj, ".plt")) != NULL) + bfd_byte *contents = splt->contents; + unsigned int i, size = splt->size; + + for (i = 0; i < size; i += 4) { - bfd_byte *contents = splt->contents; - unsigned int i, size = splt->size; - for (i = 0; i < size; i += 4) - { - unsigned int x = bfd_get_32 (dynobj, contents + i); - BFD_ASSERT (x != 0); - } + unsigned int x = bfd_get_32 (dynobj, contents + i); + BFD_ASSERT (x != 0); } }