public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] gdb: LoongArch: Support reg aliases in info reg command
@ 2023-02-23 12:37 Tiezhu Yang
  0 siblings, 0 replies; only message in thread
From: Tiezhu Yang @ 2023-02-23 12:37 UTC (permalink / raw)
  To: gdb-cvs

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

commit 0373a76af6b8ef949b9ca8aaa6325fbde0948603
Author: Hui Li <lihui@loongson.cn>
Date:   Thu Feb 16 09:22:25 2023 +0800

    gdb: LoongArch: Support reg aliases in info reg command
    
    According to LoongArch ELF ABI specification [1], support the register
    aliases in "info register" command.
    
    Without this patch:
    ```
    (gdb) info reg a0
    Invalid register `a0'
    
    ```
    With this patch:
    
    ```
    (gdb) info reg a0
    
    a0             0x1                 1
    
    ```
    [1] https://loongson.github.io/LoongArch-Documentation/LoongArch-ELF-ABI-EN.html#_register_convention
    
    Signed-off-by: Hui Li <lihui@loongson.cn>
    Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>

Diff:
---
 gdb/loongarch-tdep.c | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/gdb/loongarch-tdep.c b/gdb/loongarch-tdep.c
index 67ea5494d90..f5ddad0ba65 100644
--- a/gdb/loongarch-tdep.c
+++ b/gdb/loongarch-tdep.c
@@ -387,6 +387,14 @@ loongarch_software_single_step (struct regcache *regcache)
   return {next_pc};
 }
 
+/* Callback function for user_reg_add.  */
+
+static struct value *
+value_of_loongarch_user_reg (frame_info_ptr frame, const void *baton)
+{
+  return value_of_register ((long long) baton, frame);
+}
+
 /* Implement the frame_align gdbarch method.  */
 
 static CORE_ADDR
@@ -1589,6 +1597,19 @@ loongarch_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
   info.target_desc = tdesc;
   info.tdesc_data = tdesc_data.get ();
 
+  for (int i = 0; i < ARRAY_SIZE (loongarch_r_lp64_name); ++i)
+    if (loongarch_r_lp64_name[i][0] != '\0')
+      user_reg_add (gdbarch, loongarch_r_lp64_name[i] + 1,
+	value_of_loongarch_user_reg, (void *) (size_t) i);
+
+  for (int i = 0; i < ARRAY_SIZE (loongarch_f_lp64_name); ++i)
+    {
+      if (loongarch_f_lp64_name[i][0] != '\0')
+	user_reg_add (gdbarch, loongarch_f_lp64_name[i] + 1,
+		      value_of_loongarch_user_reg,
+		      (void *) (size_t) (LOONGARCH_FIRST_FP_REGNUM + i));
+    }
+
   /* Information about registers.  */
   set_gdbarch_num_regs (gdbarch, regnum);
   set_gdbarch_sp_regnum (gdbarch, LOONGARCH_SP_REGNUM);

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-02-23 12:37 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-23 12:37 [binutils-gdb] gdb: LoongArch: Support reg aliases in info reg command Tiezhu Yang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).