[PATCH][GAS][ARM] Gas erroneously accepts ld<cc>s[hb] mnemonics

Richard Earnshaw rearnsha@arm.com
Thu Oct 11 15:27:00 GMT 2012


On 03/10/12 12:28, Kyrylo Tkachov wrote:
> Sorry, the previous message was messed up by my email client.
> Hello,
> Gas accepts the condition codes for the ldsh and ldsb after the second
> character when they should be
> after the third one (i.e. lds<cc>[hb] instead of ld<cc>s[hb]). This patch
> fixes that and removes the unused
> macros that remain after the change.
> Ok for trunk?
>
> Thanks,
> Kyrill
>
> gas/
>
> 2012-10-03  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
>
> 	* config/tc-arm.c: Change condition code insertion for
> 	lds[hb] instructions from after the 2nd character to after the 3rd.
> 	(tCM): Remove macro.
> 	(TxCM): Likewise.
> 	(TxCM_): Likewise.
> 	(TCM): Likewise.
>
> gas/testsuite
>
> 2012-10-03  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
>
> 	* gas/arm/ldgesb-bad.d: New file.
> 	* gas/arm/ldgesb-bad.l: Likewise.
> 	* gas/arm/ldgesb-bad.s: Likewise.
> 	* gas/arm/ldgesh-bad.d: Likewise.
> 	* gas/arm/ldgesh-bad.l: Likewise.
> 	* gas/arm/ldgesh-bad.s: Likewise.
> 	* gas/arm/ldsgeb.d: Likewise.
> 	* gas/arm/ldsgeb.s: Likewise.
> 	* gas/arm/ldsgeb.l: Likewise.
> 	* gas/arm/ldsgeh.d: Likewise.
> 	* gas/arm/ldsgeh.s: Likewise.
> 	* gas/arm/ldsgeh.l: Likewise.
>
>
Thanks, I've put this in.

R.



More information about the Binutils mailing list