diff mbox series

Remove two unused fields from gdbarch

Message ID 20230117194612.3433270-1-tromey@adacore.com
State Committed
Headers
Series Remove two unused fields from gdbarch |

Commit Message

Tom Tromey Jan. 17, 2023, 7:46 p.m. UTC 
  When I converted gdbarch to use the registry, I forgot to remove the
two fields that were used to implement the previous approach.  This
patch removes them.  Tested by rebuilding.
---
 gdb/gdbarch.c  | 4 ----
 gdb/gdbarch.py | 4 ----
 2 files changed, 8 deletions(-)

Comments

Simon Marchi Jan. 17, 2023, 7:59 p.m. UTC | #1 
On 1/17/23 14:46, Tom Tromey via Gdb-patches wrote:
> When I converted gdbarch to use the registry, I forgot to remove the
> two fields that were used to implement the previous approach.  This
> patch removes them.  Tested by rebuilding.

LGTM (seems obvious even).

Simon
diff mbox series

Patch

diff --git a/gdb/gdbarch.c b/gdb/gdbarch.c
index 46baca9c479..04fcc92f8f9 100644
--- a/gdb/gdbarch.c
+++ b/gdb/gdbarch.c
@@ -48,10 +48,6 @@  struct gdbarch
   gdbarch_tdep_up tdep;
   gdbarch_dump_tdep_ftype *dump_tdep = nullptr;
 
-  /* per-architecture data-pointers.  */
-  unsigned nr_data = 0;
-  void **data = nullptr;
-
   int short_bit = 2*TARGET_CHAR_BIT;
   int int_bit = 4*TARGET_CHAR_BIT;
   int long_bit = 4*TARGET_CHAR_BIT;
diff --git a/gdb/gdbarch.py b/gdb/gdbarch.py
index 7dc79386940..3ebc3598047 100755
--- a/gdb/gdbarch.py
+++ b/gdb/gdbarch.py
@@ -289,10 +289,6 @@  with open("gdbarch.c", "w") as f:
     print("  gdbarch_tdep_up tdep;", file=f)
     print("  gdbarch_dump_tdep_ftype *dump_tdep = nullptr;", file=f)
     print(file=f)
-    print("  /* per-architecture data-pointers.  */", file=f)
-    print("  unsigned nr_data = 0;", file=f)
-    print("  void **data = nullptr;", file=f)
-    print(file=f)
     for c in filter(not_info, components):
         if isinstance(c, Function):
             print(f"  gdbarch_{c.name}_ftype *", file=f, end="")