[Patch ARM] Support MVFR2 VFP Coprocessor register for ARMv8-A,*

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[Patch ARM] Support MVFR2 VFP Coprocessor register for ARMv8-A,*

Ramana Radhakrishnan-8
Hi

        This patch adds support mvfr2 control registers for armv8-a as this was
missed from the original port to armv8-a (documented at G6.2.109 in
(Issue B.a) of the ARM-ARM. This was discovered by an internal user of
the GNU toolchain.

I'd like to backport this to the binutils 2.28 and binutils 2.29 release
branch if possible (with suitable testing and basically checking
removing the armv8-r parts).

Tristan - are you ok with the backports ?

Applied to trunk.

regards
Ramana



2017-07-04  Ramana Radhakrishnan  <[hidden email]>

         * gas/config/tc-arm.c (arm_regs): Add MVFR2.
         (do_vmrs): Constraint for MVFR2 and armv8.
         (do_vmsr): Likewise.
         * gas/testsuite/gas/arm/armv8-a+fp.d: Update.
         * gas/testsuite/gas/arm/armv8-ar+fp.s: Likewise.
         * gas/testsuite/gas/arm/armv8-r+fp.d: Likewise.
         * gas/testsuite/gas/arm/vfp-bad.s: Likewise.
         * gas/testsuite/gas/arm/vfp-bad.l: Likewise.
         * opcodes/arm-dis.c: Support MVFR2 in disassembly
         with vmrs and vmsr.
Reply | Threaded
Open this post in threaded view
|

Re: [Patch ARM] Support MVFR2 VFP Coprocessor register for ARMv8-A,*

Ramana Radhakrishnan-8
On 7/4/17 4:16 PM, Ramana Radhakrishnan wrote:

> Hi
>
>      This patch adds support mvfr2 control registers for armv8-a as this
> was missed from the original port to armv8-a (documented at G6.2.109 in
> (Issue B.a) of the ARM-ARM. This was discovered by an internal user of
> the GNU toolchain.
>
> I'd like to backport this to the binutils 2.28 and binutils 2.29 release
> branch if possible (with suitable testing and basically checking
> removing the armv8-r parts).
>
> Tristan - are you ok with the backports ?
>
> Applied to trunk.
>
> regards
> Ramana
>
>
>
> 2017-07-04  Ramana Radhakrishnan  <[hidden email]>
>
>          * gas/config/tc-arm.c (arm_regs): Add MVFR2.
>          (do_vmrs): Constraint for MVFR2 and armv8.
>          (do_vmsr): Likewise.
>          * gas/testsuite/gas/arm/armv8-a+fp.d: Update.
>          * gas/testsuite/gas/arm/armv8-ar+fp.s: Likewise.
>          * gas/testsuite/gas/arm/armv8-r+fp.d: Likewise.
>          * gas/testsuite/gas/arm/vfp-bad.s: Likewise.
>          * gas/testsuite/gas/arm/vfp-bad.l: Likewise.
>          * opcodes/arm-dis.c: Support MVFR2 in disassembly
>          with vmrs and vmsr.
And now with patch attached as what I just pushed.


Ramana

gas.txt (8K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [Patch ARM] Support MVFR2 VFP Coprocessor register for ARMv8-A,*

Tristan Gingold-2
In reply to this post by Ramana Radhakrishnan-8
On 04/07/2017 17:16, Ramana Radhakrishnan wrote:

> Hi
>
>      This patch adds support mvfr2 control registers for armv8-a as this
> was missed from the original port to armv8-a (documented at G6.2.109 in
> (Issue B.a) of the ARM-ARM. This was discovered by an internal user of
> the GNU toolchain.
>
> I'd like to backport this to the binutils 2.28 and binutils 2.29 release
> branch if possible (with suitable testing and basically checking
> removing the armv8-r parts).
>
> Tristan - are you ok with the backports ?

Yes, that's ok.
Reply | Threaded
Open this post in threaded view
|

Re: [Patch ARM] Support MVFR2 VFP Coprocessor register for ARMv8-A,*

Ramana Radhakrishnan-8
On 7/5/17 8:20 AM, Tristan Gingold wrote:

> On 04/07/2017 17:16, Ramana Radhakrishnan wrote:
>> Hi
>>
>>      This patch adds support mvfr2 control registers for armv8-a as
>> this was missed from the original port to armv8-a (documented at
>> G6.2.109 in (Issue B.a) of the ARM-ARM. This was discovered by an
>> internal user of the GNU toolchain.
>>
>> I'd like to backport this to the binutils 2.28 and binutils 2.29
>> release branch if possible (with suitable testing and basically
>> checking removing the armv8-r parts).
>>
>> Tristan - are you ok with the backports ?
>
> Yes, that's ok.


Now pushed to the branches after testing on an arm-none-eabi cross build.

regards
Ramana