------- 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
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils

Reply via email to