This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: gas/i386/inval-equ-2 test
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: "Jan Beulich" <jbeulich at novell dot com>
- Cc: hongjiu dot lu at intel dot com, binutils at sourceware dot org
- Date: Thu, 13 Nov 2008 07:48:53 -0800
- Subject: Re: gas/i386/inval-equ-2 test
- References: <491BF2E8.76E4.0078.0@novell.com>
On Thu, Nov 13, 2008 at 12:27 AM, Jan Beulich <jbeulich@novell.com> wrote:
> H.J.,
>
> I'm trying to understand what this is really trying to verify - due to a
> supposedly unrelated change this test is failing for me. I'm seeing two
> possible problems with the test itself, though:
>
> For one, (%eax+1) can't really be considered a register symbol - I really
> think that it's an error for the assembler to keep the resulting symbol in
> reg_section (this should hold at least for all targets where
> md_register_arithmetic is zero). That's also one of my local changes - to
> force such things into expr_section.
>
> Secondly, using the equated symbols in an instruction implies that you
> expect the parser to happily accept the (invalid) construct, which I think
> is not the purpose of the test (and which also fails with my local change,
> because [validly] only reg_section symbols are accepted by
> parse_register()).
>
> Hence I would think that either the expressions ought to be changed to
> have plain registers on the right side, or the move instructions ought to
> be removed.
>
> Looking forward to read your opinion,
I added it to avoid assembler crash. I don't mind you change it as long as
assembler doesn't crash and does reasonable things.
--
H.J.