[binutils-gdb] fr30: use is_whitespace()

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


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

commit 21c901a95d0e97ac5064f6a5ece039ebc7087e43
Author: Jan Beulich <jbeulich@suse.com>
Date:   Mon Feb 3 12:01:59 2025 +0100

    fr30: use is_whitespace()
    
    Convert open-coded checks. At the same time use is_end_of_stmt() instead
    of an open-coded check in adjacent code.

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

diff --git a/gas/config/tc-fr30.c b/gas/config/tc-fr30.c
index 7d1944db27d..2b135f45f00 100644
--- a/gas/config/tc-fr30.c
+++ b/gas/config/tc-fr30.c
@@ -359,7 +359,7 @@ fr30_is_colon_insn (char *start, char *nul_char)
     {
       /* Nope - check to see a 'd' follows the colon.  */
       if (   (i_l_p[1] == 'd' || i_l_p[1] == 'D')
-	  && (i_l_p[2] == ' ' || i_l_p[2] == '\t' || i_l_p[2] == '\n'))
+	  && (is_whitespace (i_l_p[2]) || is_end_of_stmt (i_l_p[2])))
 	{
 	  /* Yup - it might be delay slot instruction.  */
 	  int           i;
@@ -393,17 +393,17 @@ fr30_is_colon_insn (char *start, char *nul_char)
     }
 
   /* Check to see if the text following the colon is '8'.  */
-  if (i_l_p[1] == '8' && (i_l_p[2] == ' ' || i_l_p[2] == '\t'))
+  if (i_l_p[1] == '8' && is_whitespace (i_l_p[2]))
     return restore_colon (i_l_p + 2, nul_char);
 
   /* Check to see if the text following the colon is '20'.  */
   else if (i_l_p[1] == '2' && i_l_p[2] =='0'
-	   && (i_l_p[3] == ' ' || i_l_p[3] == '\t'))
+	   && is_whitespace (i_l_p[3]))
     return restore_colon (i_l_p + 3, nul_char);
 
   /* Check to see if the text following the colon is '32'.  */
   else if (i_l_p[1] == '3' && i_l_p[2] =='2'
-	   && (i_l_p[3] == ' ' || i_l_p[3] == '\t'))
+	   && is_whitespace (i_l_p[3]))
     return restore_colon (i_l_p + 3, nul_char);
 
   return 0;


More information about the Binutils-cvs mailing list