diff options
author | Cary Coutant <ccoutant@gmail.com> | 2015-07-21 12:42:07 -0700 |
---|---|---|
committer | Cary Coutant <ccoutant@gmail.com> | 2015-07-21 12:42:07 -0700 |
commit | cb5cf5e26ea2e42937a2ce1b662e61e4772ba6f9 (patch) | |
tree | 4ed81d81fa24984156a4cf11d54d4b5dbb42dd00 /gold | |
parent | af1b22f3004774f8c5c570abe7fab629026032f7 (diff) | |
download | fsf-binutils-gdb-cb5cf5e26ea2e42937a2ce1b662e61e4772ba6f9.zip fsf-binutils-gdb-cb5cf5e26ea2e42937a2ce1b662e61e4772ba6f9.tar.gz fsf-binutils-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().
Diffstat (limited to 'gold')
-rw-r--r-- | gold/ChangeLog | 7 | ||||
-rw-r--r-- | gold/archive.cc | 12 | ||||
-rw-r--r-- | gold/parameters.cc | 2 |
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; } |