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: http://git.efficios.com/?a=commitdiff_plain;h=47fc0f1f211b925f68e4bb8d34b5df5479fd5596;p=deliverable%2Fbinutils-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 baff095944..cfe54f93f6 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 11d1ecc745..fb31176da0 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); } }