From: H.J. Lu Date: Sat, 29 Mar 2003 01:26:33 +0000 (+0000) Subject: 2003-03-28 H.J. Lu X-Git-Tag: binutils-2_14-branchpoint~356 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5941afc1054ede37b6d8981269b8fe4519458eac;p=external%2Fbinutils.git 2003-03-28 H.J. Lu * elflink.h (elf_link_add_object_symbols): Correctly combine visibilities. --- diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 9079137..5d387c0 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,8 @@ +2003-03-28 H.J. Lu + + * elflink.h (elf_link_add_object_symbols): Correctly combine + visibilities. + 2003-03-27 Jakub Jelinek * elfxx-ia64.c (elfNN_ia64_relax_section): Reset self_dtpmod_offset diff --git a/bfd/elflink.h b/bfd/elflink.h index c39120d..3af34ee 100644 --- a/bfd/elflink.h +++ b/bfd/elflink.h @@ -1940,7 +1940,7 @@ elf_link_add_object_symbols (abfd, info) might be needed here. */ if (isym->st_other != 0) { - unsigned char hvis, symvis, other; + unsigned char hvis, symvis, other, nvis; /* Take the balance of OTHER from the definition. */ other = (definition ? isym->st_other : h->other); @@ -1949,8 +1949,14 @@ elf_link_add_object_symbols (abfd, info) /* Combine visibilities, using the most constraining one. */ hvis = ELF_ST_VISIBILITY (h->other); symvis = ELF_ST_VISIBILITY (isym->st_other); + if (! hvis) + nvis = symvis; + else if (! symvis) + nvis = hvis; + else + nvis = hvis < symvis ? hvis : symvis; - h->other = other | (hvis > symvis ? hvis : symvis); + h->other = other | nvis; } /* Set a flag in the hash table entry indicating the type of