From e8cd95c71e5a6ea455f60d6428a34c621d11d3bf Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Wed, 30 Dec 2009 04:00:21 +0000 Subject: [PATCH] PR 10450 * output.h (class Output_section): Add is_entsize_zero_ field. * output.cc (Output_section::Output_section): Initialize is_entsize_zero_. (Output_section::set_entsize): If two different entsizes are requested, force it to zero. (Output_section::add_input_section): Set flags for .debug_str before updating section flags. Set entsize. (Output_section::update_flags_for_input_section): Set SHF_MERGE and SHF_STRING if all input sections have those flags. --- gold/ChangeLog | 16 +++++++++++++++- gold/output.cc | 33 ++++++++++++++++++++++++++++----- gold/output.h | 2 ++ 3 files changed, 45 insertions(+), 6 deletions(-) diff --git a/gold/ChangeLog b/gold/ChangeLog index 0af6d8f24f7..5bf395e16ad 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,7 +1,21 @@ +2009-12-29 Ian Lance Taylor + + PR 10450 + * output.h (class Output_section): Add is_entsize_zero_ field. + * output.cc (Output_section::Output_section): Initialize + is_entsize_zero_. + (Output_section::set_entsize): If two different entsizes are + requested, force it to zero. + (Output_section::add_input_section): Set flags for .debug_str + before updating section flags. Set entsize. + (Output_section::update_flags_for_input_section): Set SHF_MERGE + and SHF_STRING if all input sections have those flags. + 2009-12-29 Rafael Espindola * main.cc (main): Fix the sys time reporting. - * workqueue.cc (Workqueue::find_and_run_task): Fix the sys time reporting. + * workqueue.cc (Workqueue::find_and_run_task): Fix the sys time + reporting. 2009-12-29 Sriraman Tallam diff --git a/gold/output.cc b/gold/output.cc index 440d4f2b579..d75b45bfd22 100644 --- a/gold/output.cc +++ b/gold/output.cc @@ -1800,6 +1800,7 @@ Output_section::Output_section(const char* name, elfcpp::Elf_Word type, is_interp_(false), is_dynamic_linker_section_(false), generate_code_fills_at_write_(false), + is_entsize_zero_(false), tls_offset_(0), checkpoint_(NULL), merge_section_map_(), @@ -1824,10 +1825,15 @@ Output_section::~Output_section() void Output_section::set_entsize(uint64_t v) { - if (this->entsize_ == 0) + if (this->is_entsize_zero_) + ; + else if (this->entsize_ == 0) this->entsize_ = v; - else - gold_assert(this->entsize_ == v); + else if (this->entsize_ != v) + { + this->entsize_ = 0; + this->is_entsize_zero_ = 1; + } } // Add the input section SHNDX, with header SHDR, named SECNAME, in @@ -1863,8 +1869,6 @@ Output_section::add_input_section(Sized_relobj* object, this->addralign_ = addralign; typename elfcpp::Elf_types::Elf_WXword sh_flags = shdr.get_sh_flags(); - this->update_flags_for_input_section(sh_flags); - uint64_t entsize = shdr.get_sh_entsize(); // .debug_str is a mergeable string section, but is not always so @@ -1875,6 +1879,9 @@ Output_section::add_input_section(Sized_relobj* object, entsize = 1; } + this->update_flags_for_input_section(sh_flags); + this->set_entsize(entsize); + // If this is a SHF_MERGE section, we pass all the input sections to // a Output_data_merge. We don't try to handle relocations for such // a section. We don't try to handle empty merge sections--they @@ -2199,6 +2206,22 @@ Output_section::update_flags_for_input_section(elfcpp::Elf_Xword flags) & (elfcpp::SHF_WRITE | elfcpp::SHF_ALLOC | elfcpp::SHF_EXECINSTR)); + + if ((flags & elfcpp::SHF_MERGE) == 0) + this->flags_ &=~ elfcpp::SHF_MERGE; + else + { + if (this->current_data_size_for_child() == 0) + this->flags_ |= elfcpp::SHF_MERGE; + } + + if ((flags & elfcpp::SHF_STRINGS) == 0) + this->flags_ &=~ elfcpp::SHF_STRINGS; + else + { + if (this->current_data_size_for_child() == 0) + this->flags_ |= elfcpp::SHF_STRINGS; + } } // Find the merge section into which an input section with index SHNDX in diff --git a/gold/output.h b/gold/output.h index 3b060c86584..55e278bbf24 100644 --- a/gold/output.h +++ b/gold/output.h @@ -3352,6 +3352,8 @@ class Output_section : public Output_data bool is_dynamic_linker_section_ : 1; // Whether code-fills are generated at write. bool generate_code_fills_at_write_ : 1; + // Whether the entry size field should be zero. + bool is_entsize_zero_ : 1; // For SHT_TLS sections, the offset of this section relative to the base // of the TLS segment. uint64_t tls_offset_; -- 2.30.2