Fix gold to group sections correctly via linker script.
authorCary Coutant <ccoutant@gmail.com>
Thu, 4 Jun 2015 02:11:42 +0000 (19:11 -0700)
committerCary Coutant <ccoutant@gmail.com>
Thu, 4 Jun 2015 02:37:49 +0000 (19:37 -0700)
commit374082dfab280123f5a54a23b1c1b2cb893b4d2b
treeee46d25beff71515f332061bc9865c0ab5c822db
parenta96a7cce1820951c566cfe793ccc72db1e4adf69
Fix gold to group sections correctly via linker script.

In PR 15370, it is noted that gold does not distinguish between
"*(.foo .bar)" and "*(.foo) *(.bar)" in linker scripts. In both
cases, gold groups all .foo sections together, followed by all
.bar sections, whereas in the first case, it should collect all
.foo and .bar sections in the order seen.

If you add sort specs, the Gnu linker has some bizarre corner
cases that I do not try to replicate. In particular,
"*(SORT_BY_NAME(.foo) SORT_BY_NAME(.bar))" does the same thing
as "*(.foo) *(.bar)". But if you apply a sort spec
to just one of several patterns, say, "*(SORT_BY_NAME(.foo) .bar)",
the Gnu linker will collect any .bar section it sees before the
first .foo, then all .foo sections, then all remaining .bar
sections. With this patch, if any of the input patterns have a
sort spec, gold will group them all as it did before; e.g.,
all .foo sections followed by all .bar sections.

2015-06-03  Cary Coutant  <ccoutant@gmail.com>

gold/
PR gold/15370
* script-sections.cc
(Output_section_element_input::set_section_addresses): When there
are several patterns with no sort spec, put all sections in the same
bin.
* testsuite/Makefile.am (script_test_12): New testcase.
(script_test_12i): New testcase.
* testsuite/Makefile.in: Regenerate.
* testsuite/script_test_12.t: New test linker script.
* testsuite/script_test_12i.t: New test linker script.
* testsuite/script_test_12a.c: New test source file.
* testsuite/script_test_12b.c: New test source file.
gold/ChangeLog
gold/script-sections.cc
gold/testsuite/Makefile.am
gold/testsuite/Makefile.in
gold/testsuite/script_test_12.t [new file with mode: 0644]
gold/testsuite/script_test_12a.c [new file with mode: 0644]
gold/testsuite/script_test_12b.c [new file with mode: 0644]
gold/testsuite/script_test_12i.t [new file with mode: 0644]