Re: vax binutils 2.39 gas problems

[Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>]

> 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