This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 2/2, AArch64] change the gas verbose error reporting on by default


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]