This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [patch] misassembly of NEON logic ops
- From: Richard Earnshaw <rearnsha at arm dot com>
- To: Paul Brook <paul at codesourcery dot com>
- Cc: binutils at sourceware dot org
- Date: Thu, 07 Sep 2006 09:51:14 +0100
- Subject: Re: [patch] misassembly of NEON logic ops
- References: <200609061732.25646.paul@codesourcery.com>
On Wed, 2006-09-06 at 17:32, Paul Brook wrote:
> The attached patch fixes misassembly of some NEON logic ops.
> vand d1, d2, d3
> would be incorrectly encoded as
> vand d1, d1, d3
>
> The operand parsing routineparse two operands at once, but doesn't mark the
> first as present. This means the encoding routine thinks we are dealing the
> two-operand form.
>
> Tested with cross to arm-none-eabi.
> Ok?
>
> Paul
>
> 2006-09-06 Paul Brook <paul@codesourcery.com>
>
> gas/
> * config/tc-arm.c (parse_operands): Mark operand as present.
>
> gas/testsuite/
> * gas/arm/neon-omit.s: Test three-argument variants.
> * gas/arm/neon-omit.d: Update expected output.
OK.
R.