This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH 2/2, AArch64] change the gas verbose error reporting on by default
- From: Marcus Shawcroft <marcus dot shawcroft at gmail dot com>
- To: Jiong Wang <jiong dot wang at arm dot com>, "binutils at sourceware dot org" <binutils at sourceware dot org>
- Date: Tue, 18 Mar 2014 10:24:36 +0000
- Subject: Re: [PATCH 2/2, AArch64] change the gas verbose error reporting on by default
- Authentication-results: sourceware.org; auth=none
- References: <5321A558 dot 4090509 at arm dot com>
On 13 March 2014 12:32, Jiong Wang <jiong.wang@arm.com> wrote:
> This patch set verbose error default on for AArch64, and basically, only
> SIMD error report affected. I also run through gcc, gdb testsuite, no
> testcase affected.
>
> Please review.
>
> Thanks
> Jiong
>
> gas/
>
> * config/tc-aarch64.c (aarch64_opts): Add new option "mno-verbose-error".
> (verbose_error_p): Initialize to 1.
> * doc/c-aarch64.texi (AArch64 Options): Document -mverbose-error and
> -mno-verbose-error.
>
> gas/testsuite/
>
> * gas/aarch64/illegal.d: Pass -mno-verbose-error.
> * gas/aarch64/verbose-error.s: Add more verbose message testcases.
> * gas/aarch64/verbose-error.l: Ditto.
+This option enables verbose error messages for AArch64 gas. This option
+is default on for AArch64 backend.
+
How about rewording the last sentence as:
"This option is enabled by default."
Otherwise, this is OK.
/Marcus