This is the mail archive of the
gdb-cvs@sourceware.org
mailing list for the GDB project.
[binutils-gdb] gdb: Prefer RISC-V register name "s0" over "fp"
- From: Sebastian Huber <sh at sourceware dot org>
- To: gdb-cvs at sourceware dot org
- Date: 2 Jul 2018 13:29:33 -0000
- Subject: [binutils-gdb] gdb: Prefer RISC-V register name "s0" over "fp"
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=41823f29a811bb250ae274652281a6294fdc2530
commit 41823f29a811bb250ae274652281a6294fdc2530
Author: Sebastian Huber <sebastian.huber@embedded-brains.de>
Date: Fri Jun 29 10:50:11 2018 +0200
gdb: Prefer RISC-V register name "s0" over "fp"
The "fp" register name is an alias for "s0" which is an alias for "x8".
The "fp" name is only understood by very recent Binutils and thus not
used by GCC. GCC does not emit a frame pointer with common optimization
options such as -Og or -O2.
It is still possible to use the "fp" register name, e.g.
(gdb) p/x $fp
$1 = 0x800367c8
works.
However, in the register dump you see now:
(gdb) info registers
...
t2 0xffffffffffffffff 18446744073709551615
s0 0x800367c8 0x800367c8
s1 0x80033280 2147693184
...
gdb/
* riscv-tdep.c (riscv_register_aliases): Swap "fp" and "s0"
entries.
Diff:
---
gdb/ChangeLog | 4 ++++
gdb/riscv-tdep.c | 2 +-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index e326785..0f601bd 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2018-07-02 Sebastian Huber <sebastian.huber@embedded-brains.de>
+
+ * riscv-tdep.c (riscv_register_aliases): Swap "fp" and "s0" entries.
+
2018-06-29 Joel Brobecker <brobecker@adacore.com>
* amd64-darwin-tdep.c (x86_darwin_init_abi_64): Add missing
diff --git a/gdb/riscv-tdep.c b/gdb/riscv-tdep.c
index 1545671..4c68ef7 100644
--- a/gdb/riscv-tdep.c
+++ b/gdb/riscv-tdep.c
@@ -128,8 +128,8 @@ static const struct register_alias riscv_register_aliases[] =
{ "t0", 5 },
{ "t1", 6 },
{ "t2", 7 },
- { "fp", 8 },
{ "s0", 8 },
+ { "fp", 8 },
{ "s1", 9 },
{ "a0", 10 },
{ "a1", 11 },