------- Additional Comments From nickc at redhat dot com 2008-05-31 16:34
-------
Hi Evandro,
Thanks for the patch. I have applied it together with the changelog entry
below. Plus I added a line about it to the ld/NEWS file, and I created a linker
testsuite test for it so that we can be sure that the option continues to work.
Interestingly the new test fails for the cr16-elf and crx-elf ports, but I
could not determine why this was happening.
Cheers
Nick
ld/ChangeLog
2008-05-31 Evandro Menezes <[EMAIL PROTECTED]>
PR ld/6430
* ld.h (enum sort_order): New.
* ldlang.c (lang_check: Fix comment.
(lang_common): Sort commons in ascending or descending order.
(lang_one_common): Likewise.
* lexsup.c (ld_options): Have --sort-common take an option
argument.
(parse_args): Handle argument to --sort-common.
* ld.texinfo (--sort-common): Document new optional argument.
* NEWS: Mention new feature.
ld/testsuite/ChangeLog
2008-05-31 Nick Clifton <[EMAIL PROTECTED]>
PR ld/6430
* ld-elfcomm/elfcomm.exp (test_sort_common): Test the
ascending/descending argument to the --sort-common command line
option.
* ld-elfcomm/sort-common.s: New file.
--
What |Removed |Added
----------------------------------------------------------------------------
Status|WAITING |RESOLVED
Resolution| |FIXED
http://sourceware.org/bugzilla/show_bug.cgi?id=6430
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
_______________________________________________
bug-binutils mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/bug-binutils
