This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: The current assembler is broken
- From: Alexandre Oliva <aoliva at redhat dot com>
- To: "H. J. Lu" <hjl at lucon dot org>
- Cc: binutils at sources dot redhat dot com
- Date: Thu, 15 Mar 2007 02:23:41 -0300
- Subject: Re: The current assembler is broken
- References: <20070314195415.GA21752@lucon.org>
On Mar 14, 2007, "H. J. Lu" <hjl@lucon.org> wrote:
> See
> http://sources.redhat.com/bugzilla/show_bug.cgi?id=4184
Thanks for the report and testcase. This patch fixes it. I could
reintroduce a state to output the blank, but... No point in doing it
if we don't need it, right?
Ok to install?
for gas/ChangeLog
from Alexandre Oliva <aoliva@redhat.com>
* app.c (do_scrub_chars): PUT after setting states.
Index: trunk/gas/app.c
===================================================================
--- trunk.orig/gas/app.c 2007-03-09 20:25:22.000000000 -0300
+++ trunk/gas/app.c 2007-03-15 02:14:23.000000000 -0300
@@ -510,11 +510,10 @@ do_scrub_chars (int (*get) (char *, int)
ch = GET ();
if (ch == '"')
{
- PUT (' ');
- PUT (ch);
quotechar = ch;
state = 5;
old_state = 3;
+ PUT (ch);
}
else
{
--
Alexandre Oliva http://www.lsd.ic.unicamp.br/~oliva/
FSF Latin America Board Member http://www.fsfla.org/
Red Hat Compiler Engineer aoliva@{redhat.com, gcc.gnu.org}
Free Software Evangelist oliva@{lsd.ic.unicamp.br, gnu.org}