[binutils-gdb] bfd: partial revert commit EC1ACAB (prevent all but undef weak symbols to become dynamic in sparc).

[Jose E.Marchesi <jemarch at sourceware dot org>]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=d8cd8c1f482018acd16cab1f1f2a081786a04c6f

commit d8cd8c1f482018acd16cab1f1f2a081786a04c6f
Author: Egeyar Bagcioglu <egeyar.bagcioglu@oracle.com>
Date:   Mon Jul 3 20:06:35 2017 +0200

    bfd: partial revert commit EC1ACAB (prevent all but undef weak symbols to become dynamic in sparc).
    
    bfd/ChangeLog:
    
    2017-07-03  Egeyar Bagcioglu  <egeyar.bagcioglu@oracle.com>
    
            * elfxx-sparc.c (_bfd_sparc_elf_finish_dynamic_symbol): Remove the
            abort statement that was put for symbols that are not dynamic.

Diff:
---
 bfd/ChangeLog     | 5 +++++
 bfd/elfxx-sparc.c | 7 -------
 2 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 06a58c1..a2b79d9 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,8 @@
+2017-07-03  Egeyar Bagcioglu  <egeyar.bagcioglu@oracle.com>
+
+        * elfxx-sparc.c (_bfd_sparc_elf_finish_dynamic_symbol): Remove the
+        abort statement that was put for symbols that are not dynamic.
+
 2017-07-03  Tristan Gingold  <gingold@adacore.com>
 
 	* po/bfd.pot: Regenerate
diff --git a/bfd/elfxx-sparc.c b/bfd/elfxx-sparc.c
index 1fd2141..b50d34b 100644
--- a/bfd/elfxx-sparc.c
+++ b/bfd/elfxx-sparc.c
@@ -4502,13 +4502,6 @@ _bfd_sparc_elf_finish_dynamic_symbol (bfd *output_bfd,
 
   eh = (struct _bfd_sparc_elf_link_hash_entry *) h;
 
-  /* Abort if the symbol is not dynamic in PIC */
-  if (h->dynindx == -1
-      && !h->forced_local
-      && h->root.type != bfd_link_hash_undefweak
-      && bfd_link_pic (info))
-    abort();
-
   /* We keep PLT/GOT entries without dynamic PLT/GOT relocations for
      resolved undefined weak symbols in executable so that their
      references have value 0 at run-time.  */