[PATCH v4 3/3] gas: Add macro tests for PR gas/32073

Jan Beulich jbeulich@suse.com
Tue Aug 13 15:12:47 GMT 2024


On 12.08.2024 17:43, H.J. Lu wrote:
> --- /dev/null
> +++ b/gas/testsuite/gas/macros/arg1.s
> @@ -0,0 +1,12 @@
> +# Test expression argument with inner white spaces and a white space
> +# before argument added by C preprocessor.
> +
> +	.macro test arg1, arg2, arg3
> +	.byte \arg1
> +	.byte \arg2
> +	.byte \ arg3
> +	.endm
> +
> +	.data
> + 	test  0x10 + 0, 0x10 + 1, 0x10 + 2
> + 	test  0x10 + 3, 0x10 + 4, 0x15

Just to have mentioned it in public: If the goal really is to (long term)
retain the ability to use such expressions with inner whitespace and
without parentheses or quotes, then reverting a few more changes done in
preparation may be desirable:

a1b7023447d1 bfin: correct macro use in gas testsuite
2231ac9b9e88 ia64: correct macro use in gas testsuite
c0e9aca554e3 MIPS: correct macro use in gas and ld testsuites
6e892703fedb TilePro: correct macro use in gas testsuite

One might think that cfa18744d435 ("Arm: correct macro use in gas
testsuite") should also fall into this group, but imo the situation is
a little different there.

Jan


More information about the Binutils mailing list