This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
[GOLD] PowerPC relaxation corner case
- From: Alan Modra <amodra at gmail dot com>
- To: binutils at sourceware dot org
- Date: Wed, 26 Nov 2014 10:31:55 +1030
- Subject: [GOLD] PowerPC relaxation corner case
- Authentication-results: sourceware.org; auth=none
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. Which is what this patch does.
* powerpc.cc (Stub_control::set_output_and_owner): New function.
(Target_powerpc::group_sections): Use it.
diff --git a/gold/powerpc.cc b/gold/powerpc.cc
index 4c90e38..2319789 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<size, big_endian>::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)
--
Alan Modra
Australia Development Lab, IBM