[PATCH] arm: don't silently accept certain wrong VST4 variants

Richard Earnshaw rearnsha@arm.com
Mon Apr 8 15:25:00 GMT 2013


On 08/04/13 14:53, Jan Beulich wrote:
> VST4 with operands matching VLD4's "single element to all lanes" assembled
> silently to opcode 0xffffffff, and other VSTn variants produced unrelated
> errors.
>
> gas/
> 2013-04-08  Jan Beulich <jbeulich@suse.com>
>
> 	* gas/config/tc-arm.c (do_neon_ldx_stx): Reject VSTn in the
> 	NEON_ALL_LANES case.
>
> --- 2013-04-08/gas/config/tc-arm.c
> +++ 2013-04-08/gas/config/tc-arm.c
> @@ -15999,6 +15999,11 @@ do_neon_ldx_stx (void)
>
>       case NEON_ALL_LANES:
>         NEON_ENCODE (DUP, inst);
> +      if (inst.instruction == N_INV)
> +	{
> +	  first_error ("only loads support such operands");
> +	  break;
> +	}
>         do_neon_ld_dup ();
>         break;
>
>
>

This needs some tests, please.  Suggest you add them to 
neon-addressing-bad.[dls]

R.




More information about the Binutils mailing list