[Bug gas/2623] No line separator character is defined for msp430-gas

2007-02-28 Thread nickc at redhat dot com
--- Additional Comments From nickc at redhat dot com 2007-02-28 10:02 --- Hi Robert, Excellent. I have applied the uploaded patch along with this ChangeLog entry. Cheers Nick gas/ChangeLog PR gas/2623 * config/tc-msp430.c (line_separator_char): Change to '{'. --

[Bug gas/2623] No line separator character is defined for msp430-gas

2007-02-28 Thread nickc at redhat dot com
--- Additional Comments From nickc at redhat dot com 2007-02-28 10:01 --- Created an attachment (id=1588) --> (http://sourceware.org/bugzilla/attachment.cgi?id=1588&action=view) Change line command character to '{' -- http://sourceware.org/bugzilla/show_bug.cgi?id=2623 --- You

[Bug gas/2623] No line separator character is defined for msp430-gas

2007-02-27 Thread rschiele at gmail dot com
--- Additional Comments From rschiele at gmail dot com 2007-02-28 04:11 --- Ok, everything of the toolchain including the jtag module built fine with '{' as separator. Since I can't see another problematic use of '{' for assembler code I think this is a reasonable choice. -- http://s

[Bug gas/2623] No line separator character is defined for msp430-gas

2007-02-27 Thread rschiele at gmail dot com
--- Additional Comments From rschiele at gmail dot com 2007-02-27 15:02 --- I will do a test build of the whole toolchain now including the jtag module to test '{' as the separator and report back then... -- http://sourceware.org/bugzilla/show_bug.cgi?id=2623 --- You are receivi

[Bug gas/2623] No line separator character is defined for msp430-gas

2007-02-27 Thread nickc at redhat dot com
--- Additional Comments From nickc at redhat dot com 2007-02-27 14:30 --- Subject: Re: No line separator character is defined for msp430-gas Hi Robert, > Well, maybe almost nobody noticed this until now because there was no official > binutils release since then. > > Actually I consi

Re: [Bug gas/2623] No line separator character is defined for msp430-gas

2007-02-27 Thread Nick Clifton
Hi Robert, Well, maybe almost nobody noticed this until now because there was no official binutils release since then. Actually I consider this to be a problem because for example the msp430-libc msp430/common.h header defines some constants to setup the watchdog. These constants are designed t

[Bug gas/2623] No line separator character is defined for msp430-gas

2007-02-27 Thread rschiele at gmail dot com
--- Additional Comments From rschiele at gmail dot com 2007-02-27 13:13 --- Well, maybe almost nobody noticed this until now because there was no official binutils release since then. Actually I consider this to be a problem because for example the msp430-libc msp430/common.h header def

[Bug gas/2623] No line separator character is defined for msp430-gas

2007-02-27 Thread nickc at redhat dot com
--- Additional Comments From nickc at redhat dot com 2007-02-27 11:44 --- Hi Robert, > Hmm, wasn't '|' known to be the bitwise OR operator? Yes. But: a) Other ports use | as the line separator character. b) The patch has been in for a while now, so changing the character w

[Bug gas/2623] No line separator character is defined for msp430-gas

2007-02-25 Thread rschiele at gmail dot com
-- What|Removed |Added CC||rschiele at gmail dot com http://sourceware.org/bugzilla/show_bug.cgi?id=2623 --- You are receiving this

[Bug gas/2623] No line separator character is defined for msp430-gas

2007-02-25 Thread rschiele at gmail dot com
--- Additional Comments From rschiele at gmail dot com 2007-02-25 19:26 --- Hmm, wasn't '|' known to be the bitwise OR operator? -- What|Removed |Added Status|RE

[Bug gas/2623] No line separator character is defined for msp430-gas

2006-05-08 Thread nickc at redhat dot com
--- Additional Comments From nickc at redhat dot com 2006-05-08 17:07 --- Hi Corentin, Sorry - the colon character (:) is used - by the macro code. See the file gas/testsuite/gas/macros/vararg.s for an example of this, and an example of a test that would fail if we applied the patch.