<div dir="ltr"><div dir="ltr">umm I apply the patch and then get the following errors,<div>error: call to undeclared function 'is_whitespace'; ISO C99 and later do not support implicit function declarations<br></div><div><br></div><div>Personally, also considering the file compatibility, maybe ISSPACE is a good way to go.  But if the above error can be resolved, in fact both look better then the old code.</div><div><br></div><div>Thanks</div><div>Nelson</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 20, 2024 at 3:13 PM Jan Beulich <<a href="mailto:jbeulich@suse.com">jbeulich@suse.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 20.08.2024 03:44, Nelson Chu wrote:<br>
> Looks good, thanks.<br>
<br>
No opinion then on is_whitespace() vs ISSPACE()?<br>
<br>
Jan<br>
<br>
> On Fri, Aug 9, 2024 at 8:55 PM Jan Beulich <<a href="mailto:jbeulich@suse.com" target="_blank">jbeulich@suse.com</a>> wrote:<br>
> <br>
>> Wherever blanks are permissible in input, tabs ought to be permissible,<br>
>> too. This is particularly relevant when -f is passed to gas (alongside<br>
>> appropriate input). Switch places already checking for tabs to use the<br>
>> macro, too. Note however that elsewhere in the file ISSPACE() is also<br>
>> used; probably the file should be consistent either way.<br>
>><br>
>> --- a/gas/config/tc-riscv.c<br>
>> +++ b/gas/config/tc-riscv.c<br>
>> @@ -2505,7 +2505,7 @@ my_getSmallExpression (expressionS *ep,<br>
>><br>
>>        /* Skip over whitespace and brackets, keeping count of the number<br>
>>          of brackets.  */<br>
>> -      while (*str == ' ' || *str == '\t' || *str == '(')<br>
>> +      while (is_whitespace (*str) || *str == '(')<br>
>>         if (*str++ == '(')<br>
>>           str_depth++;<br>
>>      }<br>
>> @@ -2534,7 +2534,7 @@ my_getSmallExpression (expressionS *ep,<br>
>>    probing_insn_operands = orig_probing;<br>
>><br>
>>    /* Match every open bracket.  */<br>
>> -  while (crux_depth > 0 && (*str == ')' || *str == ' ' || *str == '\t'))<br>
>> +  while (crux_depth > 0 && (*str == ')' || is_whitespace (*str)))<br>
>>      if (*str++ == ')')<br>
>>        crux_depth--;<br>
>><br>
>> @@ -3480,7 +3480,7 @@ riscv_ip (char *str, struct riscv_cl_ins<br>
>>               if (reg_lookup (&asarg, RCLASS_GPR, &regno))<br>
>>                 {<br>
>>                   char c = *oparg;<br>
>> -                 if (*asarg == ' ')<br>
>> +                 if (is_whitespace (*asarg))<br>
>>                     ++asarg;<br>
>><br>
>>                   /* Now that we have assembled one operand, we use the<br>
>> args<br>
>> @@ -3514,7 +3514,7 @@ riscv_ip (char *str, struct riscv_cl_ins<br>
>>                               ? RCLASS_GPR : RCLASS_FPR), &regno))<br>
>>                 {<br>
>>                   char c = *oparg;<br>
>> -                 if (*asarg == ' ')<br>
>> +                 if (is_whitespace (*asarg))<br>
>>                     ++asarg;<br>
>>                   switch (c)<br>
>>                     {<br>
>><br>
>><br>
> <br>
<br>
</blockquote></div></div>