[1/1] ld: Utilize specific digit ranges for different numeral systems

Message ID 20250107015455.34300-2-zengxiao@eswincomputing.com
State New
Headers
Series ld: Utilize specific digit ranges for different numeral systems |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_binutils_build--master-arm success Build passed
linaro-tcwg-bot/tcwg_binutils_build--master-aarch64 success Build passed
linaro-tcwg-bot/tcwg_binutils_check--master-aarch64 success Test passed
linaro-tcwg-bot/tcwg_binutils_check--master-arm success Test passed

Commit Message

Xiao Zeng Jan. 7, 2025, 1:54 a.m. UTC
  ld/ChangeLog:

        * ldlex.l: Utilize specific digit ranges for different
	numeral systems.

Signed-off-by: Xiao Zeng <zengxiao@eswincomputing.com>
---
 ld/ldlex.l | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Alan Modra Jan. 7, 2025, 6:56 a.m. UTC | #1
On Tue, Jan 07, 2025 at 09:54:55AM +0800, Xiao Zeng wrote:
> ld/ChangeLog:
> 
>         * ldlex.l: Utilize specific digit ranges for different
> 	numeral systems.

OK, thanks.

> Signed-off-by: Xiao Zeng <zengxiao@eswincomputing.com>
> ---
>  ld/ldlex.l | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/ld/ldlex.l b/ld/ldlex.l
> index a3b68718d64..ef0c5bdf425 100644
> --- a/ld/ldlex.l
> +++ b/ld/ldlex.l
> @@ -139,7 +139,7 @@ V_IDENTIFIER [*?.$_a-zA-Z\[\]\-\!\^\\]([*?.$_a-zA-Z0-9\[\]\-\!\^\\]|::)*
>  				return INT;
>  			}
>  
> -<MRI,EXPRESSION>([0-9A-Fa-f])+(H|h|X|x|B|b|O|o|D|d) {
> +<MRI,EXPRESSION>((([0-9A-Fa-f])+(H|h|X|x))|(([0-1])+(B|b))|(([0-7])+(O|o))|(([0-9])+(D|d))) {
>  				   int ibase ;
>  				   switch (yytext[yyleng - 1]) {
>  				    case 'X':
> -- 
> 2.17.1
  
Alan Modra Jan. 9, 2025, 5:51 a.m. UTC | #2
On Tue, Jan 07, 2025 at 05:26:07PM +1030, Alan Modra wrote:
> On Tue, Jan 07, 2025 at 09:54:55AM +0800, Xiao Zeng wrote:
> > ld/ChangeLog:
> > 
> >         * ldlex.l: Utilize specific digit ranges for different
> > 	numeral systems.
> 
> OK, thanks.

I pushed this for you.
  
Xiao Zeng Jan. 9, 2025, 8:43 a.m. UTC | #3
2025-01-09 14:21  Alan Modra <amodra@gmail.com> wrote:
>
>On Tue, Jan 07, 2025 at 05:26:07PM +1030, Alan Modra wrote:
>> On Tue, Jan 07, 2025 at 09:54:55AM +0800, Xiao Zeng wrote:
>> > ld/ChangeLog:
>> >
>> >         * ldlex.l: Utilize specific digit ranges for different
>> > numeral systems.
>>
>> OK, thanks.
>
>I pushed this for you. 
I don't have the permission to commit the code, so, thank you Alan~

>
>--
>Alan Modra
Thanks
Xiao Zeng
  

Patch

diff --git a/ld/ldlex.l b/ld/ldlex.l
index a3b68718d64..ef0c5bdf425 100644
--- a/ld/ldlex.l
+++ b/ld/ldlex.l
@@ -139,7 +139,7 @@  V_IDENTIFIER [*?.$_a-zA-Z\[\]\-\!\^\\]([*?.$_a-zA-Z0-9\[\]\-\!\^\\]|::)*
 				return INT;
 			}
 
-<MRI,EXPRESSION>([0-9A-Fa-f])+(H|h|X|x|B|b|O|o|D|d) {
+<MRI,EXPRESSION>((([0-9A-Fa-f])+(H|h|X|x))|(([0-1])+(B|b))|(([0-7])+(O|o))|(([0-9])+(D|d))) {
 				   int ibase ;
 				   switch (yytext[yyleng - 1]) {
 				    case 'X':