diff mbox series

[v2,30/65] MicroBlaze: use is_whitespace()

Message ID	9f2d724e-6dc3-4c32-ab1d-d8a1b1f7ba12@suse.com
State	New
Headers	DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 1DEAB3858019 Message-ID: <9f2d724e-6dc3-4c32-ab1d-d8a1b1f7ba12@suse.com> Date: Mon, 27 Jan 2025 17:26:49 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: [PATCH v2 30/65] MicroBlaze: use is_whitespace() From: Jan Beulich <jbeulich@suse.com> To: Binutils <binutils@sourceware.org> References: <2316ac5c-7870-4b46-9c80-eaecaef93a31@suse.com> Content-Language: en-US Cc: Michael Eager <eager@eagercon.com> Autocrypt: addr=jbeulich@suse.com; keydata= xsDiBFk3nEQRBADAEaSw6zC/EJkiwGPXbWtPxl2xCdSoeepS07jW8UgcHNurfHvUzogEq5xk hu507c3BarVjyWCJOylMNR98Yd8VqD9UfmX0Hb8/BrA+Hl6/DB/eqGptrf4BSRwcZQM32aZK 7Pj2XbGWIUrZrd70x1eAP9QE3P79Y2oLrsCgbZJfEwCgvz9JjGmQqQkRiTVzlZVCJYcyGGsD /0tbFCzD2h20ahe8rC1gbb3K3qk+LpBtvjBu1RY9drYk0NymiGbJWZgab6t1jM7sk2vuf0Py O9Hf9XBmK0uE9IgMaiCpc32XV9oASz6UJebwkX+zF2jG5I1BfnO9g7KlotcA/v5ClMjgo6Gl MDY4HxoSRu3i1cqqSDtVlt+AOVBJBACrZcnHAUSuCXBPy0jOlBhxPqRWv6ND4c9PH1xjQ3NP nxJuMBS8rnNg22uyfAgmBKNLpLgAGVRMZGaGoJObGf72s6TeIqKJo/LtggAS9qAUiuKVnygo 3wjfkS9A3DRO+SpU7JqWdsveeIQyeyEJ/8PTowmSQLakF+3fote9ybzd880fSmFuIEJldWxp Y2ggPGpiZXVsaWNoQHN1c2UuY29tPsJgBBMRAgAgBQJZN5xEAhsDBgsJCAcDAgQVAggDBBYC AwECHgECF4AACgkQoDSui/t3IH4J+wCfQ5jHdEjCRHj23O/5ttg9r9OIruwAn3103WUITZee e7Sbg12UgcQ5lv7SzsFNBFk3nEQQCACCuTjCjFOUdi5Nm244F+78kLghRcin/awv+IrTcIWF hUpSs1Y91iQQ7KItirz5uwCPlwejSJDQJLIS+QtJHaXDXeV6NI0Uef1hP20+y8qydDiVkv6l IreXjTb7DvksRgJNvCkWtYnlS3mYvQ9NzS9PhyALWbXnH6sIJd2O9lKS1Mrfq+y0IXCP10eS FFGg+Av3IQeFatkJAyju0PPthyTqxSI4lZYuJVPknzgaeuJv/2NccrPvmeDg6Coe7ZIeQ8Yj t0ARxu2xytAkkLCel1Lz1WLmwLstV30g80nkgZf/wr+/BXJW/oIvRlonUkxv+IbBM3dX2OV8 AmRv1ySWPTP7AAMFB/9PQK/VtlNUJvg8GXj9ootzrteGfVZVVT4XBJkfwBcpC/XcPzldjv+3 HYudvpdNK3lLujXeA5fLOH+Z/G9WBc5pFVSMocI71I8bT8lIAzreg0WvkWg5V2WZsUMlnDL9 mpwIGFhlbM3gfDMs7MPMu8YQRFVdUvtSpaAs8OFfGQ0ia3LGZcjA6Ik2+xcqscEJzNH+qh8V m5jjp28yZgaqTaRbg3M/+MTbMpicpZuqF4rnB0AQD12/3BNWDR6bmh+EkYSMcEIpQmBM51qM EKYTQGybRCjpnKHGOxG0rfFY1085mBDZCH5Kx0cl0HVJuQKC+dV2ZY5AqjcKwAxpE75MLFkr wkkEGBECAAkFAlk3nEQCGwwACgkQoDSui/t3IH7nnwCfcJWUDUFKdCsBH/E5d+0ZnMQi+G0A nAuWpQkjM1ASeQwSHEeAWPgskBQL In-Reply-To: <2316ac5c-7870-4b46-9c80-eaecaef93a31@suse.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: list Errors-To: binutils-bounces~patchwork=sourceware.org@sourceware.org
Series	gas: whitespace handling \| [v2,00/65] gas: whitespace handling [v2,01/65] gas: consolidate whitespace recognition [v2,02/65] gas/obj-.c: use is_whitespace() [v2,03/65] Alpha/EVAX: use is_whitespace() / is_end_of_stmt() [v2,04/65] arc: use is_whitespace() [v2,05/65] Arm: use is_whitespace() [v2,06/65] aarch64: use is_whitespace() [v2,07/65] avr: use is_whitespace() [v2,08/65] bfin: use is_whitespace() [v2,09/65] bpf: use is_whitespace() [v2,10/65] CR16: use is_whitespace() [v2,11/65] cris: use is_whitespace() [v2,12/65] CRx: use is_whitespace() [v2,13/65] C-Sky: use is_whitespace() [v2,14/65] d10v: use is_whitespace() [v2,15/65] d30v: use is_whitespace() [v2,16/65] dlx: use is_whitespace() [v2,17/65] Epiphany: use is_whitespace() [v2,18/65] fr30: use is_whitespace() [v2,19/65] ft32: use is_whitespace() [v2,20/65] H8/300: use is_whitespace() [v2,21/65] HP-PA: use is_whitespace() [v2,22/65] kvx: use is_whitespace() [v2,23/65] LoongArch: use is_whitespace() [v2,24/65] m32c: use is_whitespace() [v2,25/65] m32r: use is_whitespace() [v2,26/65] M68HC1x: use is_whitespace() [v2,27/65] M68k: use is_whitespace() [v2,28/65] MCore: use is_whitespace() [v2,29/65] metag: use is_whitespace() [v2,30/65] MicroBlaze: use is_whitespace() [v2,31/65] MIPS: use is_whitespace() [v2,32/65] MMIX: use is_whitespace() [v2,33/65] mn10200: use is_whitespace() [v2,34/65] mn10300: use is_whitespace() [v2,35/65] Moxie: use is_whitespace() [v2,36/65] msp430: use is_whitespace() [v2,37/65] nds32: use is_whitespace() [v2,38/65] NS32k: use is_whitespace() [v2,39/65] PDP11: use is_whitespace() [v2,40/65] PicoJava: use is_whitespace() [v2,41/65] PPC: use is_whitespace() [v2,42/65] pru: use is_whitespace() [v2,43/65] RISC-V: use is_whitespace() [v2,44/65] rl78: use is_whitespace() [v2,45/65] rx: use is_whitespace() [v2,46/65] s12z: use is_whitespace() [v2,47/65] S/390: use is_whitespace() [v2,48/65] Score: use is_whitespace() [v2,49/65] SH: use is_whitespace() [v2,50/65] Sparc: use is_whitespace() [v2,51/65] spu: use is_whitespace() [v2,52/65] C30: use is_whitespace() [v2,53/65] C4x: use is_whitespace() [v2,54/65] C54x: use is_whitespace() [v2,55/65] C6x: use is_whitespace() [v2,56/65] v850: use is_whitespace() [v2,57/65] VAX: use is_whitespace() [v2,58/65] Visium: use is_whitespace() [v2,59/65] wasm32: use is_whitespace() [v2,60/65] x86: use is_whitespace() [v2,61/65] xgate: use is_whitespace() [v2,62/65] Xtensa: use is_whitespace() [v2,63/65] Z80: use is_whitespace() [v2,64/65] Z8k: use is_whitespace() [v2,65/65] gas: suppress use of ISSPACE() / ISBLANK()

Checks

Context	Check	Description
linaro-tcwg-bot/tcwg_binutils_build--master-arm	fail	Patch failed to apply
linaro-tcwg-bot/tcwg_binutils_build--master-aarch64	fail	Patch failed to apply

Commit Message

Jan Beulich Jan. 27, 2025, 4:26 p.m. UTC

  Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert ISSPACE() uses. At the same time use
is_end_of_stmt() instead of an open-coded check in adjacent code.
---
v2: New.

Comments

Michael Eager Jan. 29, 2025, 11:56 p.m. UTC | #1

On 1/27/25 8:26 AM, Jan Beulich wrote:
> Wherever blanks are permissible in input, tabs ought to be permissible,
> too. This is particularly relevant when -f is passed to gas (alongside
> appropriate input). Also convert ISSPACE() uses. At the same time use
> is_end_of_stmt() instead of an open-coded check in adjacent code.
> ---
> v2: New.

Looks OK.

diff mbox series

Patch

--- a/gas/config/tc-microblaze.c
+++ b/gas/config/tc-microblaze.c
@@ -400,7 +400,7 @@  parse_reg (char * s, unsigned * reg)
   unsigned tmpreg = 0;
 
   /* Strip leading whitespace.  */
-  while (ISSPACE (* s))
+  while (is_whitespace (* s))
     ++ s;
 
   if (strncasecmp (s, "rpc", 3) == 0)
@@ -573,7 +573,7 @@  parse_exp (char *s, expressionS *e)
   char *new_pointer;
 
   /* Skip whitespace.  */
-  while (ISSPACE (* s))
+  while (is_whitespace (* s))
     ++ s;
 
   save = input_line_pointer;
@@ -892,12 +892,12 @@  md_assemble (char * str)
   char name[20];
 
   /* Drop leading whitespace.  */
-  while (ISSPACE (* str))
+  while (is_whitespace (* str))
     str ++;
 
   /* Find the op code end.  */
   for (op_start = op_end = str;
-       *op_end && !is_end_of_line[(unsigned char) *op_end] && *op_end != ' ';
+       !is_end_of_stmt (*op_end) && !is_whitespace (*op_end);
        op_end++)
     {
       name[nlen] = op_start[nlen];
@@ -1808,7 +1808,7 @@  md_assemble (char * str)
     }
 
   /* Drop whitespace after all the operands have been parsed.  */
-  while (ISSPACE (* op_end))
+  while (is_whitespace (* op_end))
     op_end ++;
 
   /* Give warning message if the insn has more operands than required.  */