[PATCH, GAS] No IT usage deprecation for ARMv8-M

Thomas Preudhomme thomas.preudhomme@foss.arm.com
Mon Jan 15 14:25:00 GMT 2018


TL;DR: Committed with updated ChangeLog, is it ok for binutils 2.30?

Sorry for the delay (Christmas holidays), just committed it now. I realized the 
ChangeLog entry was incomplete so I updated it:

     2018-01-15  Thomas Preud'homme  <thomas.preudhomme@arm.com>

     gas/
         * config/tc-arm.c (it_fsm_post_encode): Do not warn if targeting M
         profile architecture or if in autodetection mode.  Clarify that
         deprecation is for performance reason and concerns Armv8-A and Armv8-R.
         * testsuite/gas/arm/armv8-ar-bad.l: Adapt to new IT deprecation warning
         message.
         * testsuite/gas/arm/armv8-ar-it-bad.l: Likewise.
         * testsuite/gas/arm/sp-pc-validations-bad-t-v8a.l: Likewise.
         * testsuite/gas/arm/udf.l: Likewise.
         * testsuite/gas/arm/udf.d: Assemble for Armv8-A explicitely.

Other than that patch is unchanged.


@Tristan: is this fix ok for Binutils 2.30? It fixes 2 mistakes in an error 
message: IT is not deprecated for Armv8-M and it is only performance deprecated 
for Armv8-A and Armv8-R.

Best regards,

Thomas

On 18/12/17 13:35, Nick Clifton wrote:
> Hi Thomas,
> 
>> 2017-11-13  Thomas Preud'homme  <thomas.preudhomme@arm.com>
>>
>>      * config/tc-arm.c (it_fsm_post_encode): Do not warn if targeting M
>>      profile architecture or if in autodetection mode.
>>      * testsuite/gas/arm/udf.d: Assemble for Armv8-A explicitely.
> 
> Approved - please apply.
> 
> Cheers
>    Nick
> 
> PS.  Sorry for the delay in reviewing.
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: no_it_deprecation_v8m.patch
Type: text/x-patch
Size: 17900 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20180115/879a6207/attachment.bin>


More information about the Binutils mailing list