gas:Reject ++ and --

[Andreas Schwab]

Jakub Jelinek <jakub@redhat.com> writes:

> Even with this fix the ++/-- patch breaks previously working code.
> E.g. Linux kernel contains:
> #define MOVE_SHORTCHUNK(src, dst, offset, t0, t1) \
>         ldub    [%src - offset - 0x02], %t0; \
>         ldub    [%src - offset - 0x01], %t1; \
>         stb     %t0, [%dst - offset - 0x02]; \
>         stb     %t1, [%dst - offset - 0x01];
> ...
>         MOVE_SHORTCHUNK(o1, o0, -0x02, g2, g3)
>         MOVE_SHORTCHUNK(o1, o0, -0x04, g2, g3)
>         MOVE_SHORTCHUNK(o1, o0, -0x06, g2, g3)
>         MOVE_SHORTCHUNK(o1, o0, -0x08, g2, g3)
>         MOVE_SHORTCHUNK(o1, o0, -0x0a, g2, g3)
>         MOVE_SHORTCHUNK(o1, o0, -0x0c, g2, g3)
>         MOVE_SHORTCHUNK(o1, o0, -0x0e, g2, g3)
>         MOVE_SHORTCHUNK(o1, o0, -0x10, g2, g3)

How about putting parentheses around offset above?

Andreas.

-- 
Andreas Schwab, SuSE Labs, schwab@suse.de
SuSE Linux AG, Maxfeldstraße 5, 90409 Nürnberg, Germany
Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."