This is the mail archive of the mailing list for the binutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[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.  Which is what this patch does.

	* (Stub_control::set_output_and_owner): New function.
	(Target_powerpc::group_sections): Use it.

diff --git a/gold/ b/gold/
index 4c90e38..2319789 100644
--- a/gold/
+++ b/gold/
@@ -2399,6 +2399,14 @@ class Stub_control
   { return output_section_; }
+  void
+  set_output_and_owner(Output_section* o,
+		       const Output_section::Input_section* i)
+  {
+    this->output_section_ = o;
+    this->owner_ = i;
+  }
   typedef enum
@@ -2515,6 +2523,7 @@ Target_powerpc<size, big_endian>::group_sections(Layout* layout,
+		  stub_control.set_output_and_owner(*o, &*i);
 		  stub_table = NULL;
 	      if (stub_table == NULL)

Alan Modra
Australia Development Lab, IBM

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]