This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: PATCH: gas/4460: x86 assembler is broken
On Fri, May 04, 2007 at 09:32:54AM +0930, Alan Modra wrote:
> On Thu, May 03, 2007 at 04:30:31PM -0700, H. J. Lu wrote:
> > The problem is an operand may have 2 trailing white spaces while
> > operand () in expr.c only expects one. Add --enable-checking to gas
> > triggers this problem. We can change the generic part of assembler
> > or change x86 assembler to remove the extra trailing white space.
> > Here is a patch to change x86 assembler. I will check it in if the
> > problem isn't fixed in 24 hours.
> >
> >
> >
> > H.J.
> > ---
> > gas/
> >
> > 2007-05-03 H.J. Lu <hongjiu.lu@intel.com>
> >
> > PR gas/4460
> > * config/tc-i386.c (i386_displacement): Remove trailing white
> > space.
>
> Better to catch the problem where the extra space is added, I think.
>
Thanks. I checked in the testcase.
H.J.