aboutsummaryrefslogtreecommitdiff
path: root/gold/symtab.cc
diff options
context:
space:
mode:
authorStephen Crane <sjc@immunant.com>2017-12-01 12:10:02 -0800
committerCary Coutant <ccoutant@gmail.com>2017-12-01 12:10:02 -0800
commit71739b698d47945463f183619078ff680c277f0f (patch)
tree52ab242525ddf5bbf79c77d120bab54b339d1318 /gold/symtab.cc
parente2e321740ce2e36a97be2a410cd56eebaa2304aa (diff)
downloadgdb-71739b698d47945463f183619078ff680c277f0f.zip
gdb-71739b698d47945463f183619078ff680c277f0f.tar.gz
gdb-71739b698d47945463f183619078ff680c277f0f.tar.bz2
Fix problem where undef can fail to trigger archive rescan.
If a shared library contains an undefined symbol and LTO adds a new reference to that same undefined symbol, the reference in the new object added by the plugin would not trigger a rescan of the archive containing the symbol. 2017-11-17 Stephen Crane <sjc@immunant.com> gold/ PR gold/22448 * symtab.cc (Symbol_table::add_from_object): Only rescan for undefined symbols in regular, not dynamic, objects.
Diffstat (limited to 'gold/symtab.cc')
-rw-r--r--gold/symtab.cc15
1 files changed, 8 insertions, 7 deletions
diff --git a/gold/symtab.cc b/gold/symtab.cc
index d1f71e0..e50b42c 100644
--- a/gold/symtab.cc
+++ b/gold/symtab.cc
@@ -990,7 +990,7 @@ Symbol_table::add_from_object(Object* object,
// ins.second: true if new entry was inserted, false if not.
Sized_symbol<size>* ret;
- bool was_undefined;
+ bool was_undefined_in_reg;
bool was_common;
if (!ins.second)
{
@@ -998,7 +998,7 @@ Symbol_table::add_from_object(Object* object,
ret = this->get_sized_symbol<size>(ins.first->second);
gold_assert(ret != NULL);
- was_undefined = ret->is_undefined();
+ was_undefined_in_reg = ret->is_undefined() && ret->in_reg();
// Commons from plugins are just placeholders.
was_common = ret->is_common() && ret->object()->pluginobj() == NULL;
@@ -1049,7 +1049,7 @@ Symbol_table::add_from_object(Object* object,
// it, then change it to NAME/VERSION.
ret = this->get_sized_symbol<size>(insdefault.first->second);
- was_undefined = ret->is_undefined();
+ was_undefined_in_reg = ret->is_undefined() && ret->in_reg();
// Commons from plugins are just placeholders.
was_common = ret->is_common() && ret->object()->pluginobj() == NULL;
@@ -1061,7 +1061,7 @@ Symbol_table::add_from_object(Object* object,
}
else
{
- was_undefined = false;
+ was_undefined_in_reg = false;
was_common = false;
Sized_target<size, big_endian>* target =
@@ -1105,9 +1105,10 @@ Symbol_table::add_from_object(Object* object,
ret->set_is_default();
}
- // Record every time we see a new undefined symbol, to speed up
- // archive groups.
- if (!was_undefined && ret->is_undefined())
+ // Record every time we see a new undefined symbol, to speed up archive
+ // groups. We only care about symbols undefined in regular objects here
+ // because undefined symbols only in dynamic objects should't trigger rescans.
+ if (!was_undefined_in_reg && ret->is_undefined() && ret->in_reg())
{
++this->saw_undefined_;
if (parameters->options().has_plugins())