[PATCH, binutils/ARM] Remove _S version or MSR/MRS special registers

Thomas Preudhomme thomas.preudhomme@foss.arm.com
Fri Aug 12 13:41:00 GMT 2016


Hi,

When adding support for ARMv8-M in GAS, MRS/MSR special registers were extended 
with MSP_S/MSP_NS and PSP_S/PSP_NS. While _NS versions make sense as they 
correspond to a different encoding, _S version do not. They are artifacts of 
previous version of the patch where a special CLI switch was considered to 
assemble ARMv8-M Secure code. In their current form they are a source of 
confusion because they would lead to read or write the non secure version of MSP 
and PSP when in non secure state.

Since this is early 2.27 days, we believe it to be best to remove them now 
rather than keep them in. Patch is in attachment.

ChangeLog entrie is as follows:

*** gas/ChangeLog ***

2016-08-11  Thomas Preud'homme  <thomas.preudhomme@arm.com>

         * config/tc-arm.c (v7m_psrs): Remove msp_s, MSP_S, psp_s and PSP_S
         special registers.
         * testsuite/gas/arm/archv8m-cmse-msr.s: Remove test for above special
         registers.
         * testsuite/gas/arm/archv8m-cmse-msr-base.d: Likewise.
         * testsuite/gas/arm/archv8m-cmse-msr-main.d: Likewise.
         * testsuite/gas/arm/archv8m-main-dsp-4.d: Likewise.


Is this ok for master and more importantly binutils-2_27-branch?

Best regards,

Thomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: remove_s_msr_special_regs.patch
Type: text/x-patch
Size: 5205 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20160812/14b7d13c/attachment.bin>


More information about the Binutils mailing list