Move common symbol check after bed->common_definition

bfd/

	* elflink.c (elf_link_add_object_symbols): Move common symbol
	check after bed->common_definition.

ld/

	* testsuite/ld-elf/pr21903.s (start): Removed.
	(_start): Likewise.
	(__start): Likewise.
	(main): Likewise.
	(bar): New.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 5d04571..3ed2e9e 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,8 @@
+2017-08-07  H.J. Lu  <hongjiu.lu@intel.com>
+
+       * elflink.c (elf_link_add_object_symbols): Move common symbol
+       check after bed->common_definition.
+
 2017-08-07  Alan Modra  <amodra@gmail.com>
 
        PR 21910

diff --git a/bfd/elflink.c b/bfd/elflink.c
index b89d537..424de21 100644 (file)
--- a/bfd/elflink.c
+++ b/bfd/elflink.c
@@ -4250,15 +4250,16 @@ error_free_dyn:
 
       override = FALSE;
 
-      /* Treat common symbol as undefined for --no-define-common.  */
-      if (isym->st_shndx == SHN_COMMON
-         && info->inhibit_common_definition)
-       isym->st_shndx = SHN_UNDEF;
-
       flags = BSF_NO_FLAGS;
       sec = NULL;
       value = isym->st_value;
       common = bed->common_definition (isym);
+      if (common && info->inhibit_common_definition)
+       {
+         /* Treat common symbol as undefined for --no-define-common.  */
+         isym->st_shndx = SHN_UNDEF;
+         common = FALSE;
+       }
       discarded = FALSE;
 
       bind = ELF_ST_BIND (isym->st_info);

diff --git a/ld/ChangeLog b/ld/ChangeLog
index 33fb868..96dc608 100644 (file)
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,11 @@
+2017-08-07  H.J. Lu  <hongjiu.lu@intel.com>
+
+       * testsuite/ld-elf/pr21903.s (start): Removed.
+       (_start): Likewise.
+       (__start): Likewise.
+       (main): Likewise.
+       (bar): New.
+
 2017-08-07  Alan Modra  <amodra@gmail.com>
 
        * testsuite/ld-alpha/alpha.exp: Add --hash-style=sysv to various

diff --git a/ld/testsuite/ld-elf/pr21903.s b/ld/testsuite/ld-elf/pr21903.s
index 9dbf96c..ce26b33 100644 (file)
--- a/ld/testsuite/ld-elf/pr21903.s
+++ b/ld/testsuite/ld-elf/pr21903.s
@@ -1,12 +1,8 @@
        .text
-       .global start   /* Used by SH targets.  */
-start:
-       .global _start
-_start:
-       .global __start
-__start:
-       .global main    /* Used by HPPA targets.  */
-main:
+       .global bar
+bar:
+       .byte 0
+       .data
        .dc.a foo
        .ifdef  HPUX
 foo    .comm   4