diff options
| author | Rahul Chaudhry <rahulchaudhry@google.com> | 2016-06-29 17:01:23 +0000 |
|---|---|---|
| committer | android-build-merger <android-build-merger@google.com> | 2016-06-29 17:01:23 +0000 |
| commit | 850063a5c4a41433b50cc3eeac69e4285ef5129a (patch) | |
| tree | c5d81e8da334346e9bbc6962fc5d7d50d3d80076 | |
| parent | cc160c2199d6d3d8b64dcc578ece5eb5e75a4b80 (diff) | |
| parent | fd48d3412ad6b8c1c84934e65cfea4f67213f79b (diff) | |
| download | toolchain_binutils-850063a5c4a41433b50cc3eeac69e4285ef5129a.tar.gz toolchain_binutils-850063a5c4a41433b50cc3eeac69e4285ef5129a.tar.bz2 toolchain_binutils-850063a5c4a41433b50cc3eeac69e4285ef5129a.zip | |
Revert \\"Disable the warning of a hidden symbol matches a reference to be resolved.\\" am: 4520d6bee1
am: fd48d3412a
Change-Id: I236eef08f85d32346eeb00ef7e03cc977483e1c0
| -rw-r--r-- | binutils-2.25/gold/ChangeLog | 9 | ||||
| -rw-r--r-- | binutils-2.25/gold/resolve.cc | 11 |
2 files changed, 18 insertions, 2 deletions
diff --git a/binutils-2.25/gold/ChangeLog b/binutils-2.25/gold/ChangeLog index c222155a..cd957520 100644 --- a/binutils-2.25/gold/ChangeLog +++ b/binutils-2.25/gold/ChangeLog @@ -1,3 +1,12 @@ +2016-06-28 Rahul Chaudhry <rahulchaudhry@google.com> + + Revert patch -- to be replaced by the upstream patch with tests. + + 2015-06-16 Yiran Wang <yiran@google.com> + + * resolve.cc disable the warning of a hidden symbol matches a + reference to be resolved + 2016-02-05 Sriraman Tallam <tmsriram@google.com> * icf.cc (get_rel_addend): New function. diff --git a/binutils-2.25/gold/resolve.cc b/binutils-2.25/gold/resolve.cc index 54de87d9..07dff4ad 100644 --- a/binutils-2.25/gold/resolve.cc +++ b/binutils-2.25/gold/resolve.cc @@ -279,8 +279,15 @@ Symbol_table::resolve(Sized_symbol<size>* to, && (to->visibility() == elfcpp::STV_HIDDEN || to->visibility() == elfcpp::STV_INTERNAL)) { - // it is good to be helpful, but the warning leads to build error - // for some users, so disable it if not really wanted. + // A dynamic object cannot reference a hidden or internal symbol + // defined in another object. + gold_warning(_("%s symbol '%s' in %s is referenced by DSO %s"), + (to->visibility() == elfcpp::STV_HIDDEN + ? "hidden" + : "internal"), + to->demangled_name().c_str(), + to->object()->name().c_str(), + object->name().c_str()); return; } else |