aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCary Coutant <ccoutant@gmail.com>2015-07-21 12:42:07 -0700
committerCary Coutant <ccoutant@gmail.com>2015-07-21 12:42:07 -0700
commitcb5cf5e26ea2e42937a2ce1b662e61e4772ba6f9 (patch)
tree4ed81d81fa24984156a4cf11d54d4b5dbb42dd00
parentaf1b22f3004774f8c5c570abe7fab629026032f7 (diff)
downloadgdb-cb5cf5e26ea2e42937a2ce1b662e61e4772ba6f9.zip
gdb-cb5cf5e26ea2e42937a2ce1b662e61e4772ba6f9.tar.gz
gdb-cb5cf5e26ea2e42937a2ce1b662e61e4772ba6f9.tar.bz2
Fix internal error when linking an archive library with no preceding objects.
gold/ PR gold/18696 * archive.cc (Library_base::should_include_member): Don't use entry point for relocatable links, or if target is not yet valid. * parameters.cc (Parameters::entry): Check target_valid().
-rw-r--r--gold/ChangeLog7
-rw-r--r--gold/archive.cc12
-rw-r--r--gold/parameters.cc2
3 files changed, 16 insertions, 5 deletions
diff --git a/gold/ChangeLog b/gold/ChangeLog
index becff51..b7f0a33 100644
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,10 @@
+2015-07-21 Cary Coutant <ccoutant@gmail.com>
+
+ PR gold/18696
+ * archive.cc (Library_base::should_include_member): Don't use entry
+ point for relocatable links, or if target is not yet valid.
+ * parameters.cc (Parameters::entry): Check target_valid().
+
2015-07-20 Han Shen <shenhan@google.com>
Optimize erratum 843419 fix.
diff --git a/gold/archive.cc b/gold/archive.cc
index 6d25980..cc69c40 100644
--- a/gold/archive.cc
+++ b/gold/archive.cc
@@ -138,11 +138,15 @@ Library_base::should_include_member(Symbol_table* symtab, Layout* layout,
return Library_base::SHOULD_INCLUDE_YES;
}
- if (strcmp(sym_name, parameters->entry()) == 0)
+ if (!parameters->options().relocatable())
{
- *why = "entry symbol ";
- *why += sym_name;
- return Library_base::SHOULD_INCLUDE_YES;
+ const char* entry_sym = parameters->entry();
+ if (entry_sym != NULL && strcmp(sym_name, entry_sym) == 0)
+ {
+ *why = "entry symbol ";
+ *why += sym_name;
+ return Library_base::SHOULD_INCLUDE_YES;
+ }
}
return Library_base::SHOULD_INCLUDE_UNKNOWN;
diff --git a/gold/parameters.cc b/gold/parameters.cc
index 8eab000..5ed1912 100644
--- a/gold/parameters.cc
+++ b/gold/parameters.cc
@@ -237,7 +237,7 @@ const char*
Parameters::entry() const
{
const char* ret = this->options().entry();
- if (ret == NULL)
+ if (ret == NULL && parameters->target_valid())
ret = parameters->target().entry_symbol_name();
return ret;
}