ppc64_elf_edit_opd revamp, take 2

Now that ld -r relocs are sorted by the pr17666 fix, there isn't so
much need to sort incoming relocs.

	* elf64-ppc.c (sort_r_offset): Delete.
	(ppc64_elf_edit_opd): Don't sort input relocs.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index ea307516b9b27664a3fa1ec57ef800dab88924b3..974b8d8a6e7b8cd97a2a7c8921b2aeb39524a727 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,8 @@
+2014-12-09  Alan Modra  <amodra@gmail.com>
+
+       * elf64-ppc.c (sort_r_offset): Delete.
+       (ppc64_elf_edit_opd): Don't sort input relocs.
+
 2014-12-06  Eric Botcazou  <ebotcazou@adacore.com>
 
        * config.bfd: Add Visium support.

diff --git a/bfd/elf64-ppc.c b/bfd/elf64-ppc.c
index 2e9d35a88681862f98ac6a7c083bbd365b163671..cb58df18d111908e09bde09cedd963e3b59a828a 100644 (file)
--- a/bfd/elf64-ppc.c
+++ b/bfd/elf64-ppc.c
@@ -7589,21 +7589,6 @@ dec_dynrel_count (bfd_vma r_info,
   return FALSE;
 }
 
-/* qsort comparison function sorting relocs by r_offset.  */
-
-static int
-sort_r_offset (const void *p, const void *q)
-{
-  const Elf_Internal_Rela *a = (const Elf_Internal_Rela *) p;
-  const Elf_Internal_Rela *b = (const Elf_Internal_Rela *) q;
-
-  if (a->r_offset < b->r_offset)
-    return -1;
-  else if (a->r_offset > b->r_offset)
-    return 1;
-  return 0;
-}
-
 /* Remove unused Official Procedure Descriptor entries.  Currently we
    only remove those associated with functions in discarded link-once
    sections, or weakly defined functions that have been overridden.  It
@@ -7657,7 +7642,6 @@ ppc64_elf_edit_opd (struct bfd_link_info *info)
                                            info->keep_memory);
       if (relstart == NULL)
        return FALSE;
-      qsort (relstart, sec->reloc_count, sizeof (*relstart), sort_r_offset);
 
       /* First run through the relocs to check they are sane, and to
         determine whether we need to edit this opd section.  */