[binutils-gdb] bfin: correct macro use in gas testsuite

Jan Beulich jbeulich@sourceware.org
Fri Jul 19 09:57:23 GMT 2024


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=a1b7023447d19d70bc36d71b7627f457dbfae5ce

commit a1b7023447d19d70bc36d71b7627f457dbfae5ce
Author: Jan Beulich <jbeulich@suse.com>
Date:   Fri Jul 19 11:56:20 2024 +0200

    bfin: correct macro use in gas testsuite
    
    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.

Diff:
---
 gas/testsuite/gas/bfin/allinsn16.s | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/gas/testsuite/gas/bfin/allinsn16.s b/gas/testsuite/gas/bfin/allinsn16.s
index 4a1636d674e..bf00cb5e603 100644
--- 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-cvs mailing list