public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH v2 0/3] gas: add command line option to inhibit info messages
@ 2024-07-04 15:51 Matthieu Longo
  2024-07-04 15:51 ` [PATCH v2 1/3] gas: minor reformatting in command line help and doc Matthieu Longo
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Matthieu Longo @ 2024-07-04 15:51 UTC (permalink / raw)
  To: binutils
  Cc: Richard Earnshaw, Nick Clifton, Jan Beulich, Alan Modra, Matthieu Longo

Hi,

GAS currently emits information messages for context information along warnings. These extra messages might sometimes pollute the test expectations when checking for error message patterns in stderr output.

This patch aims at providing two new flags while preserving the existing behavior if none of the options is provided.
  * --info, similar to the existing --warn flag to enable diagnostic information messages (default behavior).
  * --no-info, similar to the existing --no-warn flag to disable diagnostic information messages.

It also adds the flags to the existing documentation, and command manual.

As an example of its benefit, I used the new flag for the system register tests under gas/testsuite/gas/aarch64/sysreg testsuite, and the info message for macro expansion were removed inside the tests expectations.

Regression tested on aarch64-none-elf, and no regression found.

Ok for binutils-master?

Regards,
Matthieu.

Changes from v0:
- reorder warning options
  https://sourceware.org/pipermail/binutils/2024-June/135232.html
- rename flag_no_infos to flag_no_information.
- documentation: move --no-info and --info flags after warnings flags.
- remove from this patch's scope the tests relying on the line number.
  add a check in output_info() in AArch64 backend.
  check flag_no_information flag and quickly return if enabled, as requested in the previous revision's review.
  https://sourceware.org/pipermail/binutils/2024-June/135231.html

Changes from v1:
- move the '--info' section up to respect alphabetical order.

Matthieu Longo (3):
  gas: minor reformatting in command line help and doc
  gas: add new command line options to control diagnostic information
    messages
  aarch64: testsuite: remove macro expansion messages from expected
    error output

 gas/as.c                                      |  26 +-
 gas/as.h                                      |   5 +-
 gas/config/tc-aarch64.c                       |   3 +
 gas/doc/as.texi                               |  59 ++++-
 gas/messages.c                                |   3 +
 gas/testsuite/gas/aarch64/sve-movprfx_26.d    |   2 +-
 gas/testsuite/gas/aarch64/sve-movprfx_26.l    |  13 -
 .../aarch64/sysreg/armv8_9-a-sysregs-bad.d    |   2 +-
 .../aarch64/sysreg/armv8_9-a-sysregs-bad.l    | 250 +-----------------
 .../gas/aarch64/sysreg/illegal-sysreg-3.d     |   2 +-
 .../gas/aarch64/sysreg/illegal-sysreg-3.l     |  20 --
 .../gas/aarch64/sysreg/illegal-sysreg-8.d     |   1 +
 .../gas/aarch64/sysreg/illegal-sysreg-8.l     | 188 -------------
 .../gas/aarch64/sysreg/illegal-sysreg-8b.d    |   1 +
 .../gas/aarch64/sysreg/illegal-sysreg-8b.l    |  29 --
 15 files changed, 85 insertions(+), 519 deletions(-)

-- 
2.45.1


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2024-07-09  9:35 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-07-04 15:51 [PATCH v2 0/3] gas: add command line option to inhibit info messages Matthieu Longo
2024-07-04 15:51 ` [PATCH v2 1/3] gas: minor reformatting in command line help and doc Matthieu Longo
2024-07-04 15:51 ` [PATCH v2 2/3] gas: add new command line options to control diagnostic information messages Matthieu Longo
2024-07-04 15:51 ` [PATCH v2 3/3] aarch64: testsuite: remove macro expansion messages from expected error output Matthieu Longo
2024-07-09  7:40 ` [PATCH v2 0/3] gas: add command line option to inhibit info messages Jan Beulich
2024-07-09  9:10   ` Matthieu Longo
2024-07-09  9:35     ` Jan Beulich

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).