[Patch, AARCH64] Enable some gc-sections tests

Marcus Shawcroft marcus.shawcroft@gmail.com
Wed Mar 20 14:58:00 GMT 2013


Thanks for your patch.

On 15 March 2013 15:16, Venkataramanan Kumar
<venkataramanan.kumar@linaro.org> wrote:

> I have enabled few gc-sections tests in attached patch for aarch64 target.
> Built aarch64-none-linux-gnu and make check passes the tests now.
> if Ok for trunk, can someone apply them?

In future patch submissions please place the ChangeLog in the email
body rather than in the patch itself.

+2013-03-15  Venkataramanan Kumar  <venkataramanan.kumar@linaro.org>
+	* ld-elf/group8a.d: Remove aarch64
+	from list of targets that don't run this test case.

ChangeLog entries should if possible indicate what was changed, how
about this wording instead:

+       * ld-elf/group8a.d (notarget): Remove aarch64*-*-*.

+	* ld-elf/group8b.d: Likewise
+	* ld-elf/group9a.d: Likewise
+	* ld-elf/group9b.d: Likewise
+	* ld-elf/pr12851.d: Likewise
+	* ld-elf/pr12975.d: Likewise
+	* ld-elf/pr13177.d: Likewise
+	* ld-elf/pr13195.d: Likewise

ChangeLog entries are sentences therefore they end with a period.

 2013-03-08  Venkataramanan Kumar  <venkataramanan.kumar@linaro.org>

-        * lib/ld-lib.exp (check_gc_sections_available): Remove aarch64
+	* lib/ld-lib.exp (check_gc_sections_available): Remove aarch64

This should not be part of this patch.

Otherwise the patch itself is fine, I've committed it for you with
this ChangeLog entry:

2013-03-20  Venkataramanan Kumar  <venkataramanan.kumar@linaro.org>

       * ld-elf/group8a.d (notarget): Remove aarch64*-*-*.
       * ld-elf/group8b.d: Likewise.
       * ld-elf/group9a.d: Likewise.
       * ld-elf/group9b.d: Likewise.
       * ld-elf/pr12851.d: Likewise.
       * ld-elf/pr12975.d: Likewise.
       * ld-elf/pr13177.d: Likewise.
       * ld-elf/pr13195.d: Likewise.


More information about the Binutils mailing list