diff options
| author | Alan Modra <amodra@gmail.com> | 2014-11-26 10:06:07 +1030 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2014-11-26 10:50:33 +1030 |
| commit | a20605cf85452f051434296ddfd8ed109577bb6b (patch) | |
| tree | f54e425f77a0e65d8a0c8f2b3641c1b3f689b574 /gold | |
| parent | 38bd618497d57d118160976867397e8bb22732d8 (diff) | |
| download | gdb-a20605cf85452f051434296ddfd8ed109577bb6b.zip gdb-a20605cf85452f051434296ddfd8ed109577bb6b.tar.gz gdb-a20605cf85452f051434296ddfd8ed109577bb6b.tar.bz2 | |
[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.
Diffstat (limited to 'gold')
| -rw-r--r-- | gold/ChangeLog | 5 | ||||
| -rw-r--r-- | gold/powerpc.cc | 9 |
2 files changed, 14 insertions, 0 deletions
diff --git a/gold/ChangeLog b/gold/ChangeLog index 9f3f529..9b78b35 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,3 +1,8 @@ +2014-11-26 Alan Modra <amodra@gmail.com> + + * powerpc.cc (Stub_control::set_output_and_owner): New function. + (Target_powerpc::group_sections): Use it. + 2014-11-25 Cary Coutant <ccoutant@google.com> * binary.cc (Binary_to_elf::sized_convert): Add size to _start symbol. 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) |