diff options
author | Andrew Burgess <andrew.burgess@embecosm.com> | 2019-02-23 22:07:47 +0200 |
---|---|---|
committer | Andrew Burgess <andrew.burgess@embecosm.com> | 2019-02-26 22:57:35 +0200 |
commit | 172fb711a2336b62cf8d58bbb4b27dc71ca8c02d (patch) | |
tree | 2ed60a0f66549ca1d8da7f0102bc9910b3a071ac /gdb/features/riscv/64bit-fpu.xml | |
parent | 8791793caa9a3186d4922cc786b6344e6093be73 (diff) | |
download | binutils-gdb-172fb711a2336b62cf8d58bbb4b27dc71ca8c02d.tar.gz |
gdb/riscv: Use legacy register numbers in default target description
When the target description support was added to RISC-V, the register
numbers assigned to the fflags, frm, and fcsr control registers in the
default target descriptions didn't match the register numbers used by
GDB before the target description support was added.
What this means is that if a tools exists in the wild that is using
hard-coded register number, setup to match GDB's old numbering, then
this will have been broken (for fflags, frm, and fcsr) by the move to
target descriptions. QEMU is such a tool.
There are a couple of solutions that could be used to work around this
issue:
- The user can create their own xml description file with the
register numbers setup to match their old tool, then load this by
telling GDB 'set tdesc filename FILENAME'.
- Update their old tool to use the newer default numbering scheme, or
better yet add proper target description support to their tool.
- We could have RISC-V GDB change to maintain the old defaults.
This patch changes GDB back to using the old numbering scheme.
This change is only visible to remote targets that don't supply their
own xml description file and instead rely on GDB's default numbering.
Note that even though 32bit-cpu.xml and 64bit-cpu.xml have changed,
the corresponding .c file has not, this is because the numbering added
to the registers in the xml files is number 0, this doesn't result in
any new C code being generated .
gdb/ChangeLog:
* features/riscv/32bit-cpu.xml: Add register numbers.
* features/riscv/32bit-fpu.c: Regenerate.
* features/riscv/32bit-fpu.xml: Add register numbers.
* features/riscv/64bit-cpu.xml: Add register numbers.
* features/riscv/64bit-fpu.c: Regenerate.
* features/riscv/64bit-fpu.xml: Add register numbers.
Diffstat (limited to 'gdb/features/riscv/64bit-fpu.xml')
-rw-r--r-- | gdb/features/riscv/64bit-fpu.xml | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/gdb/features/riscv/64bit-fpu.xml b/gdb/features/riscv/64bit-fpu.xml index fd14ebced85..794854cc011 100644 --- a/gdb/features/riscv/64bit-fpu.xml +++ b/gdb/features/riscv/64bit-fpu.xml @@ -5,6 +5,10 @@ are permitted in any medium without royalty provided the copyright notice and this notice are preserved. --> +<!-- Register numbers are hard-coded in order to maintain backward + compatibility with older versions of tools that didn't use xml + register descriptions. --> + <!DOCTYPE feature SYSTEM "gdb-target.dtd"> <feature name="org.gnu.gdb.riscv.fpu"> @@ -13,7 +17,7 @@ <field name="double" type="ieee_double"/> </union> - <reg name="ft0" bitsize="64" type="riscv_double"/> + <reg name="ft0" bitsize="64" type="riscv_double" regnum="33"/> <reg name="ft1" bitsize="64" type="riscv_double"/> <reg name="ft2" bitsize="64" type="riscv_double"/> <reg name="ft3" bitsize="64" type="riscv_double"/> @@ -46,7 +50,7 @@ <reg name="ft10" bitsize="64" type="riscv_double"/> <reg name="ft11" bitsize="64" type="riscv_double"/> - <reg name="fflags" bitsize="32" type="int"/> - <reg name="frm" bitsize="32" type="int"/> - <reg name="fcsr" bitsize="32" type="int"/> + <reg name="fflags" bitsize="32" type="int" regnum="66"/> + <reg name="frm" bitsize="32" type="int" regnum="67"/> + <reg name="fcsr" bitsize="32" type="int" regnum="68"/> </feature> |