public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Jan Beulich <jbeulich@suse.com>
To: Binutils <binutils@sourceware.org>
Cc: Richard Earnshaw <rearnsha@arm.com>,
	Marcus Shawcroft <marcus.shawcroft@arm.com>
Subject: [PATCH 10/11] aarch64: relax gas testsuite whitespace expectations
Date: Fri, 28 Jun 2024 15:26:41 +0200	[thread overview]
Message-ID: <01b7e203-fed0-427e-b5b7-3695634f1f4d@suse.com> (raw)
In-Reply-To: <f12de588-f77f-41b5-a6d0-83c1eb7de05c@suse.com>

[-- Attachment #1: Type: text/plain, Size: 638 bytes --]

In a subsequent change the scrubber is going to be changed to retain
further whitespace. Test case expectations generally would better not
depend on the specific whitespace treatment by the scrubber, unless of
course a test is specifically about it. Adjust relevant test cases to
permit blanks where those will subsequently appear.
---
This is adding just the blanks that are going to be needed; imo it would
generally be better if test case expectations were, from the very
beginning, written to focus on what is being tested, rather than taking
verbatim copies of the respective tool's output.

[actual patch attached, for size reasons]

[-- Attachment #2: gas-testsuite-relax-Arm64.patch.gz --]
[-- Type: application/x-gzip, Size: 117830 bytes --]

  parent reply	other threads:[~2024-06-28 13:26 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-06-28 13:15 [PATCH 00/11] gas: scrubber adjustments Jan Beulich
2024-06-28 13:17 ` [PATCH 01/11] gas: there's no scrubber state 12 Jan Beulich
2024-06-28 13:17 ` [PATCH 02/11] gas: drop scrubber state -2 Jan Beulich
2024-06-28 13:18 ` [PATCH 03/11] gas: multi-byte warning adjustments Jan Beulich
2024-06-28 13:18 ` [PATCH 04/11] gas: don't open-code IS_WHITESPACE() / IS_NEWLINE() Jan Beulich
2024-06-28 13:19 ` [PATCH 05/11] gas: pre-init the scrubber's lex[] Jan Beulich
2024-07-01  1:45   ` Hans-Peter Nilsson
2024-07-01  6:02     ` Jan Beulich
2024-06-28 13:20 ` [PATCH 06/11] gas: re-work tic6x scrubber special case Jan Beulich
2024-06-28 13:22 ` [PATCH 07/11] gas: consistently drop trailing whitespace when scrubbing Jan Beulich
2024-06-28 13:53   ` Maciej W. Rozycki
2024-07-01  5:58     ` Jan Beulich
2024-07-01 11:05       ` Maciej W. Rozycki
2024-07-03 13:42       ` Michael Matz
2024-06-28 13:23 ` [PATCH 08/11] Arm: correct macro use in gas testsuite Jan Beulich
2024-06-28 13:24 ` [PATCH 09/11] Arm: relax gas testsuite whitespace expectations Jan Beulich
2024-06-28 13:26 ` Jan Beulich [this message]
2024-06-28 13:31 ` [PATCH WIP/RFC 11/11] gas: have scrubber retain more whitespace Jan Beulich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=01b7e203-fed0-427e-b5b7-3695634f1f4d@suse.com \
    --to=jbeulich@suse.com \
    --cc=binutils@sourceware.org \
    --cc=marcus.shawcroft@arm.com \
    --cc=rearnsha@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).