[Bug exp/21717] "print $fpscr" says "<unavailable>" until some VFP data register is printed

["cvs-commit at gcc dot gnu.org" <sourceware-bugzilla at sourceware dot org>]

https://sourceware.org/bugzilla/show_bug.cgi?id=21717

--- Comment #6 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Yao Qi <qiyao@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=02ad7fc29f824a9742f8386bcf5793b81ff798f5

commit 02ad7fc29f824a9742f8386bcf5793b81ff798f5
Author: Yao Qi <yao.qi@linaro.org>
Date:   Fri Jul 21 16:35:23 2017 +0100

    [ARM] Access FPSCR on vfpv2

    GDB can fetch or store FPSCR on vfpv3, which has 32 VFP registers, but
    fail to do so on vfpv2, which has 16 VFP registers.  GDB code is incorrect
    for vfpv2,

           else if (tdep->vfp_register_count > 0
               && regno >= ARM_D0_REGNUM
               && regno <= ARM_D0_REGNUM + tdep->vfp_register_count)

    while FPSCR register number is defined as ARM_D0_REGNUM + 32.

      ARM_D0_REGNUM,            /* VFP double-precision registers.  */
      ARM_D31_REGNUM = ARM_D0_REGNUM + 31,
      ARM_FPSCR_REGNUM,

    The code above uses "<=" rather than "<", in order to put FPSCR in the
    range, but it is only correct when tdep->vfp_register_count is 32.  On
    vpfv2, it is 16, and FPSCR is out of the range, so fetch_vfp_regs or
    store_vfp_regs are not called.

    gdb:

    2017-07-25  Yao Qi  <yao.qi@linaro.org>

        PR tdep/21717
        * arm-linux-nat.c (arm_linux_fetch_inferior_registers): Update
        condition for FPSCR.
        (arm_linux_store_inferior_registers): Likewise.

-- 
You are receiving this mail because:
You are on the CC list for the bug.