diff mbox series

[v2,45/65] rx: use is_whitespace()

Message ID	a050aa3c-c6cd-4ac4-afae-a2e5638bb93b@suse.com
State	New
Headers	DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 90C82385800F Message-ID: <a050aa3c-c6cd-4ac4-afae-a2e5638bb93b@suse.com> Date: Mon, 27 Jan 2025 17:35:32 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: [PATCH v2 45/65] rx: 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: Nick Clifton <nickc@redhat.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	success	Build passed
linaro-tcwg-bot/tcwg_binutils_check--master-arm	success	Test passed
linaro-tcwg-bot/tcwg_binutils_build--master-aarch64	success	Build passed
linaro-tcwg-bot/tcwg_binutils_check--master-aarch64	success	Test passed

Commit Message

Jan Beulich Jan. 27, 2025, 4:35 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 open-coded checks as well as ISSPACE()
uses. At the same time use is_end_of_stmt() instead of an open-coded
check in adjacent code.
---
v2: New.

diff mbox series

Patch

--- a/gas/config/rx-parse.y
+++ b/gas/config/rx-parse.y
@@ -1583,7 +1583,7 @@  rx_lex (void)
   unsigned int ci;
   char * save_input_pointer;
 
-  while (ISSPACE (*rx_lex_start)
+  while (is_whitespace (*rx_lex_start)
 	 && rx_lex_start != rx_lex_end)
     rx_lex_start ++;
 
--- a/gas/config/tc-rx.c
+++ b/gas/config/tc-rx.c
@@ -282,7 +282,8 @@  rx_include (int ignore)
   last_char = find_end_of_line (filename, false);
   input_line_pointer = last_char;
 
-  while (last_char >= filename && (* last_char == ' ' || * last_char == '\n'))
+  while (last_char >= filename
+	 && (is_whitespace (* last_char) || is_end_of_stmt (* last_char)))
     -- last_char;
   end_char = *(++ last_char);
   * last_char = 0;
@@ -425,14 +426,14 @@  parse_rx_section (char * name)
 	{
 	  *p = end_char;
 
-	  if (end_char == ' ')
-	    while (ISSPACE (*p))
+	  if (is_whitespace (end_char))
+	    while (is_whitespace (*p))
 	      p++;
 
 	  if (*p == '=')
 	    {
 	      ++ p;
-	      while (ISSPACE (*p))
+	      while (is_whitespace (*p))
 		p++;
 	      switch (*p)
 		{
@@ -517,7 +518,7 @@  rx_section (int ignore)
     {
       int len = p - input_line_pointer;
 
-      while (ISSPACE (*++p))
+      while (is_whitespace (*++p))
 	;
 
       if (*p != '"' && *p != '#')
@@ -1060,7 +1061,7 @@  rx_equ (char * name, char * expression)
   char * name_end;
   char * saved_ilp;
 
-  while (ISSPACE (* name))
+  while (is_whitespace (* name))
     name ++;
 
   for (name_end = name + 1; *name_end; name_end ++)
@@ -1094,7 +1095,7 @@  scan_for_infix_rx_pseudo_ops (char * str
     return false;
 
   /* A real pseudo-op must be preceded by whitespace.  */
-  if (dot[-1] != ' ' && dot[-1] != '\t')
+  if (!is_whitespace (dot[-1]))
     return false;
 
   pseudo_op = dot + 1;