This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

FIx ld-elf/multibss1.s on ARM


The recent patch to fix \@ macro expansion on ARM uncovered a failure in 
ld-elf/multibss1.s. The patch below changes it to use %nobits instead of 
@nobits, like the rest of the testsuite. The latter doesn't work because @ is 
a comment character on ARM.

Previously it worked by chance because the \@ was also treated as a comment, 
skipping the problematic code altogether.

Tested on arm-linux-gnueabi.
Applied to cvs head.

Paul

2007-05-17  Paul Brook  <paul@codesourcery.com>

	ld/testsuite/
	* ld-elf/multibss1.s: Use %nobits instead of @nobits.

Index: ld/testsuite/ld-elf/multibss1.s
===================================================================
RCS file: /var/cvsroot/src-cvs/src/ld/testsuite/ld-elf/multibss1.s,v
retrieving revision 1.1
diff -u -p -r1.1 multibss1.s
--- ld/testsuite/ld-elf/multibss1.s	10 May 2007 14:46:48 -0000	1.1
+++ ld/testsuite/ld-elf/multibss1.s	17 May 2007 16:46:23 -0000
@@ -1,5 +1,5 @@
 	.macro	makebss
-	.section .bss_\@,"aw",@nobits
+	.section .bss_\@,"aw",%nobits
 	.space	0x10000
 	.endm
 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]