[PATCH, GAS, ARM]. Warn on Erratum 752419.

Nick Clifton nickc@redhat.com
Wed May 11 09:28:00 GMT 2011


Hi Tejas,

> 	* config/tc-arm.c(do_t_ldst): Warn on loading into sp with
> 	writeback for appropriate cores/arch.
> 	* testsuite/gas/arm/ld-sp-warn-cortex-m3.d: New test.
> 	* testsuite/gas/arm/ld-sp-warn-cortex-m3.l: New test.
> 	* testsuite/gas/arm/ld-sp-warn-cortex-m4.d: New test.
> 	* testsuite/gas/arm/ld-sp-warn-cortex-m4.l: New test.
> 	* testsuite/gas/arm/ld-sp-warn-v7.d: New test.
> 	* testsuite/gas/arm/ld-sp-warn-v7.l: New test.
> 	* testsuite/gas/arm/ld-sp-warn-v7a.d: New test.
> 	* testsuite/gas/arm/ld-sp-warn-v7a.l: New test.
> 	* testsuite/gas/arm/ld-sp-warn-v7e-m.l: New test.
> 	* testsuite/gas/arm/ld-sp-warn-v7em.d: New test.
> 	* testsuite/gas/arm/ld-sp-warn-v7m.d: New test.
> 	* testsuite/gas/arm/ld-sp-warn-v7m.l: New test.
> 	* testsuite/gas/arm/ld-sp-warn-v7r.d: New test.
> 	* testsuite/gas/arm/ld-sp-warn-v7r.l: New test.
> 	* testsuite/gas/arm/ld-sp-warn.s: New test.

Approved and applied.  I made two changes however - firstly there is a 
separate ChangeLog for gas testsuite additions, so I moved your 
changelog entry there.  Secondly all of your new tests had the same text 
in their "name:" entry, so if one of them failed it would not be 
immediately obvious which one was at fault.  Thus I added a architecture 
name in parentheses to the text so that you can tell them apart.

Cheers
   Nick




More information about the Binutils mailing list