[Patch, AARCH64] Enable some gc-sections tests
Marcus Shawcroft
marcus.shawcroft@gmail.com
Wed Mar 20 14:58:00 GMT 2013
Hi,
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.
Cheers
/Marcus
More information about the Binutils
mailing list