From 2c38906f72b07a9e3b8386d9de8a61b33db04d99 Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Wed, 9 Apr 2008 00:26:48 +0000 Subject: * layout.h (class Layout): Add added_eh_frame_data_ field. * layout.cc (Layout::Layout): Initialize new field. (Layout::layout_eh_frame): Don't add eh_frame_data_ to .eh_frame output section until we find a section we merged successfully. * object.cc (Sized_relobj::check_eh_frame_flags): Don't require that the size be non-zero. --- gold/layout.cc | 18 +++++++++++++++--- 1 file changed, 15 insertions(+), 3 deletions(-) (limited to 'gold/layout.cc') diff --git a/gold/layout.cc b/gold/layout.cc index e8582ca..efeaf65 100644 --- a/gold/layout.cc +++ b/gold/layout.cc @@ -82,7 +82,8 @@ Layout::Layout(const General_options& options, Script_options* script_options) unattached_section_list_(), special_output_list_(), section_headers_(NULL), tls_segment_(NULL), symtab_section_(NULL), dynsym_section_(NULL), dynamic_section_(NULL), dynamic_data_(NULL), - eh_frame_section_(NULL), eh_frame_data_(NULL), eh_frame_hdr_section_(NULL), + eh_frame_section_(NULL), eh_frame_data_(NULL), + added_eh_frame_data_(false), eh_frame_hdr_section_(NULL), build_id_note_(NULL), group_signatures_(), output_file_size_(-1), input_requires_executable_stack_(false), input_with_gnu_stack_note_(false), @@ -563,7 +564,6 @@ Layout::layout_eh_frame(Sized_relobj* object, { this->eh_frame_section_ = os; this->eh_frame_data_ = new Eh_frame(); - os->add_output_section_data(this->eh_frame_data_); if (this->options_.eh_frame_hdr()) { @@ -605,7 +605,19 @@ Layout::layout_eh_frame(Sized_relobj* object, shndx, reloc_shndx, reloc_type)) - *off = -1; + { + // We found a .eh_frame section we are going to optimize, so now + // we can add the set of optimized sections to the output + // section. We need to postpone adding this until we've found a + // section we can optimize so that the .eh_frame section in + // crtbegin.o winds up at the start of the output section. + if (!this->added_eh_frame_data_) + { + os->add_output_section_data(this->eh_frame_data_); + this->added_eh_frame_data_ = true; + } + *off = -1; + } else { // We couldn't handle this .eh_frame section for some reason. -- cgit v1.1