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

Jiong Wang jiong.wang@arm.com
Thu Mar 13 12:32:00 GMT 2014


For AArch64 advanced SIMD assembly coding,  there are some constraints 
on operand formats. For some instructions, only specific subtypes are 
accepted.

Take "fcvtn" for example,

without verbose error
==

./as-new -mno-verbose-error  -o 2.o 2.s

2.s: Assembler messages:
2.s:1: Error: operand mismatch -- `fcvtn v25.4s,v26.4s'

with verbose error
==
./as-new -mverbose-error -o 2.o 2.s
2.s: Assembler messages:
2.s:1: Error: operand mismatch -- `fcvtn v25.4s,v26.4s'
2.s:1: Info:    did you mean this?
2.s:1: Info:        fcvtn v25.4h,v26.4s
2.s:1: Info:    other valid variant(s):
2.s:1: Info:        fcvtn v25.2s,v26.2d

More heuristic and helpful messages are delivered to user.

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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: set-verbose-error-default-on.patch
Type: text/x-patch
Size: 9783 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20140313/14ad1cf1/attachment.bin>


More information about the Binutils mailing list