[binutils-gdb] SH: use is_whitespace()

Jan Beulich jbeulich@sourceware.org
Mon Feb 3 11:23:56 GMT 2025


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=f483688fd004382f83e887ed78e80558d36bc67c

commit f483688fd004382f83e887ed78e80558d36bc67c
Author: Jan Beulich <jbeulich@suse.com>
Date:   Mon Feb 3 12:22:10 2025 +0100

    SH: use is_whitespace()
    
    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 an
    ISSPACE() use. At the same time use is_end_of_stmt() instead of
    (kind-of-)open-coded checks in adjacent code.

Diff:
---
 gas/config/tc-sh.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/gas/config/tc-sh.c b/gas/config/tc-sh.c
index d1a7c2516ec..850e8382b72 100644
--- a/gas/config/tc-sh.c
+++ b/gas/config/tc-sh.c
@@ -1231,7 +1231,7 @@ get_operands (sh_opcode_info *info, char *args, sh_operand_info *operand)
       /* The pre-processor will eliminate whitespace in front of '@'
 	 after the first argument; we may be called multiple times
 	 from assemble_ppi, so don't insist on finding whitespace here.  */
-      if (*ptr == ' ')
+      if (is_whitespace (*ptr))
 	ptr++;
 
       get_operand (&ptr, operand + 0);
@@ -2150,7 +2150,7 @@ find_cooked_opcode (char **str_p)
   unsigned int nlen = 0;
 
   /* Drop leading whitespace.  */
-  while (*str == ' ')
+  while (is_whitespace (*str))
     str++;
 
   /* Find the op code end.
@@ -2158,9 +2158,8 @@ find_cooked_opcode (char **str_p)
      any '@' after the first argument; we may be called from
      assemble_ppi, so the opcode might be terminated by an '@'.  */
   for (op_start = op_end = (unsigned char *) str;
-       *op_end
-       && nlen < sizeof (name) - 1
-       && !is_end_of_line[*op_end] && *op_end != ' ' && *op_end != '@';
+       nlen < sizeof (name) - 1
+       && !is_end_of_stmt (*op_end) && !is_whitespace (*op_end) && *op_end != '@';
        op_end++)
     {
       unsigned char c = op_start[nlen];
@@ -2514,10 +2513,11 @@ md_assemble (char *str)
       bool found = false;
 
       /* Identify opcode in string.  */
-      while (ISSPACE (*name))
+      while (is_whitespace (*name))
 	name++;
 
-      while (name[name_length] != '\0' && !ISSPACE (name[name_length]))
+      while (!is_end_of_stmt (name[name_length])
+	     && !is_whitespace (name[name_length]))
 	name_length++;
 
       /* Search for opcode in full list.  */
@@ -2576,7 +2576,7 @@ md_assemble (char *str)
 	    {
 	      /* Ignore trailing whitespace.  If there is any, it has already
 		 been compressed to a single space.  */
-	      if (*op_end == ' ')
+	      if (is_whitespace (*op_end))
 		op_end++;
 	    }
 	  else


More information about the Binutils-cvs mailing list