This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
RE: [PATCH] gdb/testsuite/gdb.dwarf2: Replaces @ with % sign to allow tests stay compatible with both arm and x86 assembly
- From: "Abid, Hafiz" <Hafiz_Abid at mentor dot com>
- To: Omair Javaid <omair dot javaid at linaro dot org>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Mon, 10 Jun 2013 13:08:04 +0000
- Subject: RE: [PATCH] gdb/testsuite/gdb.dwarf2: Replaces @ with % sign to allow tests stay compatible with both arm and x86 assembly
- References: <CANW4E-25m0GrfpR7XTYVRYOKwkpXgLvtMnNdriaR_DkPM=cNfQ at mail dot gmail dot com>
Hi Omair,
> -----Original Message-----
> From: gdb-patches-owner@sourceware.org [mailto:gdb-patches-
> owner@sourceware.org] On Behalf Of Omair Javaid
> Sent: 10 June 2013 02:18
> To: gdb-patches@sourceware.org
> Subject: [PATCH] gdb/testsuite/gdb.dwarf2: Replaces @ with % sign to allow
> tests stay compatible with both arm and x86 assembly
>
> ARM assembler recognise @ symbol as a comment marker and therefore is
> unable to assemble some of gdb.dwarf2 testsuite tests. This patch replaces
> all occurrences of @ symbol with % sign to allow tests stay compatible with
> both arm and x86 assembly. This patch has been tested on x86_64 and
> armv7 targets.
>
> gdb/testsuite/ChangeLog:
>
> 2013-06-10 Omair Javaid <Omair.Javaid@linaro.org>
>
> * gdb.dwarf2/fission-reread.S: Replace @ sign with % sign to remove
> assembler errors on arm.
> * gdb.dwarf2/mac-fileno.S: Likewise
> * gdb.dwarf2/member-ptr-forwardref.S: Likewise
> * gdb.dwarf2/pr13961.S: Likewise
Just a small nit. Please indent ChangeLog with tab. Also in ChangeLog, you describe what was changed and not why was it changed. So "to remove assembler errors on arm" above may be unnecessary.
Regards,
Abid