[PATCH v2 08/17] bfin: correct macro use in gas testsuite

Jan Beulich jbeulich@suse.com
Fri Jul 12 13:03:45 GMT 2024


Whitespace in macro arguments either needs quoting / parenthesizing to
reliably not be mistaken for an argument separator, or respective macro
parameters need to be marked as covering all remaining arguments. The
latter really isn't an option here.
---
v2: New.

--- a/gas/testsuite/gas/bfin/allinsn16.s
+++ b/gas/testsuite/gas/bfin/allinsn16.s
@@ -12,16 +12,16 @@
 # iterate 0x20 times
 .macro _dw b, i, e
 	.if \i < \e
-		__dw \b, \i, \i + 0x40
-		_dw \b, \i + 0x40, \e
+		__dw \b, \i, (\i + 0x40)
+		_dw \b, (\i + 0x40), \e
 	.endif
 .endm
 
 # iterate 0x4 times
 .macro dw b, i, e
 	.if \i < \e
-		_dw \b, \i, \i + 0x800
-		dw \b, \i + 0x800, \e
+		_dw \b, \i, (\i + 0x800)
+		dw \b, (\i + 0x800), \e
 	.endif
 .endm
 



More information about the Binutils mailing list