From 5dc824ed42cd173c1525f5abc76f4091f11a4dbc Mon Sep 17 00:00:00 2001 From: Cary Coutant Date: Tue, 28 Nov 2017 18:48:12 -0800 Subject: [PATCH] Allow multiply-defined absolute symbols when they have the same value. gold/ * resolve.cc (Symbol_table::resolve): Allow multiply-defined absolute symbols when they have the same value. --- gold/ChangeLog | 5 +++++ gold/resolve.cc | 20 +++++++++++++++----- 2 files changed, 20 insertions(+), 5 deletions(-) diff --git a/gold/ChangeLog b/gold/ChangeLog index a87b99d..f158a10 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,5 +1,10 @@ 2017-11-28 Cary Coutant + * resolve.cc (Symbol_table::resolve): Allow multiply-defined absolute + symbols when they have the same value. + +2017-11-28 Cary Coutant + * object.h (class Sized_relobj_file): Remove discarded_eh_frame_shndx_. * object.cc (Sized_relobj_file::Sized_relobj_file): Likewise. (Sized_relobj_file::layout_eh_frame_section): Likewise. diff --git a/gold/resolve.cc b/gold/resolve.cc index d447ef0..ae8197d 100644 --- a/gold/resolve.cc +++ b/gold/resolve.cc @@ -247,18 +247,28 @@ Symbol_table::resolve(Sized_symbol* to, Object* object, const char* version, bool is_default_version) { + bool to_is_ordinary; + const unsigned int to_shndx = to->shndx(&to_is_ordinary); + // It's possible for a symbol to be defined in an object file // using .symver to give it a version, and for there to also be // a linker script giving that symbol the same version. We // don't want to give a multiple-definition error for this // harmless redefinition. - bool to_is_ordinary; if (to->source() == Symbol::FROM_OBJECT && to->object() == object - && is_ordinary && to->is_defined() - && to->shndx(&to_is_ordinary) == st_shndx + && is_ordinary && to_is_ordinary + && to_shndx == st_shndx + && to->value() == sym.get_st_value()) + return; + + // Likewise for an absolute symbol defined twice with the same value. + if (!is_ordinary + && st_shndx == elfcpp::SHN_ABS + && !to_is_ordinary + && to_shndx == elfcpp::SHN_ABS && to->value() == sym.get_st_value()) return; @@ -350,8 +360,8 @@ Symbol_table::resolve(Sized_symbol* to, && (sym.get_st_bind() == elfcpp::STB_WEAK || to->binding() == elfcpp::STB_WEAK) && orig_st_shndx != elfcpp::SHN_UNDEF - && to->shndx(&to_is_ordinary) != elfcpp::SHN_UNDEF && to_is_ordinary + && to_shndx != elfcpp::SHN_UNDEF && sym.get_st_size() != 0 // Ignore weird 0-sized symbols. && to->symsize() != 0 && (sym.get_st_type() != to->type() @@ -362,7 +372,7 @@ Symbol_table::resolve(Sized_symbol* to, { Symbol_location fromloc = { object, orig_st_shndx, static_cast(sym.get_st_value()) }; - Symbol_location toloc = { to->object(), to->shndx(&to_is_ordinary), + Symbol_location toloc = { to->object(), to_shndx, static_cast(to->value()) }; this->candidate_odr_violations_[to->name()].insert(fromloc); this->candidate_odr_violations_[to->name()].insert(toloc); -- 2.7.4