Re: vax binutils 2.39 gas problems

[Christos Zoulas <christos%zoulas.com@localhost>]

I think I found the problem....

christos

> On Jan 27, 2023, at 12:19 PM, Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost> wrote:
> 
> 
>> 
>> Hmm. Unless I'm getting this wrong, the error comes from the assembly 
>> phase. So could you compare the .s files with and without those extra 
>> spaces for the last line of the _asm() ?
> 
> Here it is:
> --- loadfile-fail.s    2023-01-28 01:06:37.114441994 +0900
> +++ loadfile-work.s    2023-01-28 01:06:12.131622723 +0900
> @@ -96,7 +96,7 @@
>    rotl $-8, %r0, %r2 
> insv %r2, $16, $8, %r2 
> rotl $8, %r0, %r1 
> -movb %r1, %r2
> +movb %r1, %r2  
> # 0 "" 2
> #NO_APP
>    cmpw %r2,$263
> @@ -106,7 +106,7 @@
>    rotl $-8, %r0, %r2 
> insv %r2, $16, $8, %r2 
> rotl $8, %r0, %r1 
> -movb %r1, %r2
> +movb %r1, %r2  
> # 0 "" 2
> #NO_APP
>    cmpw %r2,$264
> ---
> 
>> But yeah, sounds like gas wants some whitespace after the last character 
>> to be happy. Newline or spaces, or at least something. (I'm sort of 
>> wondering if it somehow sucks in the next line, containing "# 0 "" 2" 
>> and that's where the complaints about the missing closing quote comes from.)
> 
> Using NetBSD/vax 9.3 tools (gcc 7.5.0 and binutils-2.31.1) also shows:
> ---
> :
> 
> .L10:
>    movl -56(%fp),%r0
>    bitl %r0,$-65536
>    jeql .L12
> #APP
> # 47 "../.././machine/byte_swap.h" 1
>    rotl $-8, %r0, %r2 
> insv %r2, $16, $8, %r2 
> rotl $8, %r0, %r1 
> movb %r1, %r2
> # 0 "" 2
> #NO_APP
>    cmpw %r2,$263
>    jeql .L13
> #APP
> # 47 "../.././machine/byte_swap.h" 1
>    rotl $-8, %r0, %r2 
> insv %r2, $16, $8, %r2 
> rotl $8, %r0, %r1 
> movb %r1, %r2
> # 0 "" 2
> #NO_APP
>    cmpw %r2,$264
>    jneq .L9
> 
> :
> ---
> 
>> 
>> Also, do you get the same result if you just run gas on the .s files 
>> directly?
> 
> Yes.
> 
> ---
> % /s/cvs/src/obj.vax/tooldir.NetBSD-9.3-i386/bin/vax--netbsdelf-as -v
> GNU assembler version 2.39 (vax--netbsdelf) using BFD version (NetBSD Binutils nb1) 2.39
> % /s/cvs/src/obj.vax/tooldir.NetBSD-9.3-i386/bin/vax--netbsdelf-as loadfile-fail.s
> loadfile-fail.s: Assembler messages:
> loadfile-fail.s:99: Warning: missing closing `"'
> loadfile-fail.s:99: Fatal error: Ignoring statement due to "Unknown operator"
> % /s/cvs/src/obj.vax/tooldir.NetBSD-9.3-i386/bin/vax--netbsdelf-as loadfile-work.s
> % 
> ---
> Izumi Tsutsui