From a20605cf85452f051434296ddfd8ed109577bb6b Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Wed, 26 Nov 2014 10:06:07 +1030 Subject: [PATCH] [GOLD] PowerPC relaxation corner case It's possible for the section grouping code to decide that the last section looked at (the first section by address) doesn't fit into a group. In one case that section is already a group owner and that is handled correctly. In other cases the section should be put into its own group. Like this: * powerpc.cc (Stub_control::set_output_and_owner): New function. (Target_powerpc::group_sections): Use it. --- gold/ChangeLog | 5 +++++ gold/powerpc.cc | 9 +++++++++ 2 files changed, 14 insertions(+) diff --git a/gold/ChangeLog b/gold/ChangeLog index 9f3f529659c..9b78b35a3e6 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,3 +1,8 @@ +2014-11-26 Alan Modra + + * powerpc.cc (Stub_control::set_output_and_owner): New function. + (Target_powerpc::group_sections): Use it. + 2014-11-25 Cary Coutant * binary.cc (Binary_to_elf::sized_convert): Add size to _start symbol. diff --git a/gold/powerpc.cc b/gold/powerpc.cc index 4c90e389b85..231978910e7 100644 --- a/gold/powerpc.cc +++ b/gold/powerpc.cc @@ -2399,6 +2399,14 @@ class Stub_control output_section() { return output_section_; } + void + set_output_and_owner(Output_section* o, + const Output_section::Input_section* i) + { + this->output_section_ = o; + this->owner_ = i; + } + private: typedef enum { @@ -2515,6 +2523,7 @@ Target_powerpc::group_sections(Layout* layout, { stub_table->init(stub_control.owner(), stub_control.output_section()); + stub_control.set_output_and_owner(*o, &*i); stub_table = NULL; } if (stub_table == NULL) -- 2.30.2