[patch] misassembly of NEON logic ops

Richard Earnshaw rearnsha@arm.com
Thu Sep 7 08:51:00 GMT 2006


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.



More information about the Binutils mailing list