From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2126) id 1C3D03864804; Tue, 18 Jan 2022 17:33:04 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 1C3D03864804 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Tom Tromey To: gdb-cvs@sourceware.org Subject: [binutils-gdb] Introduce gdb-hashtab module in gdbsupport X-Act-Checkin: binutils-gdb X-Git-Author: Tom Tromey X-Git-Refname: refs/heads/master X-Git-Oldrev: bf31fd38f02ca9b1a7d75e2d00ee0af665fd3efd X-Git-Newrev: 0589ca4e7ba9b8d60599706b57be22c007c1f4fa Message-Id: <20220118173304.1C3D03864804@sourceware.org> Date: Tue, 18 Jan 2022 17:33:04 +0000 (GMT) X-BeenThere: gdb-cvs@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 18 Jan 2022 17:33:04 -0000 https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3D0589ca4e7ba9= b8d60599706b57be22c007c1f4fa commit 0589ca4e7ba9b8d60599706b57be22c007c1f4fa Author: Tom Tromey Date: Tue Dec 21 16:48:38 2021 -0700 Introduce gdb-hashtab module in gdbsupport =20 gdb has some extensions and helpers for working with the libiberty hash table. This patch consolidates these and moves them to gdbsupport. Diff: --- gdb/compile/compile-internal.h | 1 + gdb/completer.h | 1 + gdb/filename-seen-cache.h | 1 + gdb/gdbtypes.h | 1 + gdb/utils.c | 24 -------------------- gdb/utils.h | 26 ---------------------- gdbsupport/Makefile.am | 1 + gdbsupport/Makefile.in | 20 +++++++++-------- gdbsupport/gdb-hashtab.cc | 44 +++++++++++++++++++++++++++++++++++++ gdbsupport/gdb-hashtab.h | 50 ++++++++++++++++++++++++++++++++++++++= ++++ 10 files changed, 110 insertions(+), 59 deletions(-) diff --git a/gdb/compile/compile-internal.h b/gdb/compile/compile-internal.h index 92efd511075..042ddf9c5f9 100644 --- a/gdb/compile/compile-internal.h +++ b/gdb/compile/compile-internal.h @@ -18,6 +18,7 @@ #define COMPILE_COMPILE_INTERNAL_H =20 #include "gcc-c-interface.h" +#include "gdbsupport/gdb-hashtab.h" =20 /* Debugging flag for the "compile" family of commands. */ =20 diff --git a/gdb/completer.h b/gdb/completer.h index 93125e4c54c..78e5240a95e 100644 --- a/gdb/completer.h +++ b/gdb/completer.h @@ -17,6 +17,7 @@ #if !defined (COMPLETER_H) #define COMPLETER_H 1 =20 +#include "gdbsupport/gdb-hashtab.h" #include "gdbsupport/gdb_vecs.h" #include "command.h" =20 diff --git a/gdb/filename-seen-cache.h b/gdb/filename-seen-cache.h index 09529031973..83a6e8a6258 100644 --- a/gdb/filename-seen-cache.h +++ b/gdb/filename-seen-cache.h @@ -22,6 +22,7 @@ =20 #include "defs.h" #include "gdbsupport/function-view.h" +#include "gdbsupport/gdb-hashtab.h" =20 /* Cache to watch for file names already seen. */ =20 diff --git a/gdb/gdbtypes.h b/gdb/gdbtypes.h index 467f60975e1..7238873e4db 100644 --- a/gdb/gdbtypes.h +++ b/gdb/gdbtypes.h @@ -46,6 +46,7 @@ =20 #include "hashtab.h" #include "gdbsupport/array-view.h" +#include "gdbsupport/gdb-hashtab.h" #include "gdbsupport/gdb_optional.h" #include "gdbsupport/offset-type.h" #include "gdbsupport/enum-flags.h" diff --git a/gdb/utils.c b/gdb/utils.c index 0be49a6d4f8..bda3b7b5f0d 100644 --- a/gdb/utils.c +++ b/gdb/utils.c @@ -2841,30 +2841,6 @@ gdb_argv_as_array_view_test () =20 #endif /* GDB_SELF_TEST */ =20 -/* Allocation function for the libiberty hash table which uses an - obstack. The obstack is passed as DATA. */ - -void * -hashtab_obstack_allocate (void *data, size_t size, size_t count) -{ - size_t total =3D size * count; - void *ptr =3D obstack_alloc ((struct obstack *) data, total); - - memset (ptr, 0, total); - return ptr; -} - -/* Trivial deallocation function for the libiberty splay tree and hash - table - don't deallocate anything. Rely on later deletion of the - obstack. DATA will be the obstack, although it is not needed - here. */ - -void -dummy_obstack_deallocate (void *object, void *data) -{ - return; -} - /* Simple, portable version of dirname that does not modify its argument. */ =20 diff --git a/gdb/utils.h b/gdb/utils.h index 5ac34eb41a9..8e9c2049915 100644 --- a/gdb/utils.h +++ b/gdb/utils.h @@ -123,27 +123,6 @@ extern int parse_escape (struct gdbarch *, const char = **); =0C /* Cleanup utilities. */ =20 -/* A deleter for a hash table. */ -struct htab_deleter -{ - void operator() (htab *ptr) const - { - htab_delete (ptr); - } -}; - -/* A unique_ptr wrapper for htab_t. */ -typedef std::unique_ptr htab_up; - -/* A wrapper for 'delete' that can used as a hash table entry deletion - function. */ -template -void -htab_delete_entry (void *ptr) -{ - delete (T *) ptr; -} - extern void init_page_info (void); =20 /* Temporarily set BATCH_FLAG and the associated unlimited terminal size. @@ -384,11 +363,6 @@ extern void demangler_warning (const char *file, int l= ine, =0C /* Misc. utilities. */ =20 -/* Allocation and deallocation functions for the libiberty hash table - which use obstacks. */ -void *hashtab_obstack_allocate (void *data, size_t size, size_t count); -void dummy_obstack_deallocate (void *object, void *data); - #ifdef HAVE_WAITPID extern pid_t wait_to_die_with_timeout (pid_t pid, int *status, int timeout= ); #endif diff --git a/gdbsupport/Makefile.am b/gdbsupport/Makefile.am index 73d4149f294..9e4334891b5 100644 --- a/gdbsupport/Makefile.am +++ b/gdbsupport/Makefile.am @@ -52,6 +52,7 @@ libgdbsupport_a_SOURCES =3D \ filestuff.cc \ format.cc \ gdb-dlfcn.cc \ + gdb-hashtab.cc \ gdb_obstack.cc \ gdb_tilde_expand.cc \ gdb_wait.cc \ diff --git a/gdbsupport/Makefile.in b/gdbsupport/Makefile.in index 88d3dd153d6..59e1dfedd8b 100644 --- a/gdbsupport/Makefile.in +++ b/gdbsupport/Makefile.in @@ -151,15 +151,15 @@ am_libgdbsupport_a_OBJECTS =3D agent.$(OBJEXT) btrace= -common.$(OBJEXT) \ common-regcache.$(OBJEXT) common-utils.$(OBJEXT) \ environ.$(OBJEXT) errors.$(OBJEXT) event-loop.$(OBJEXT) \ fileio.$(OBJEXT) filestuff.$(OBJEXT) format.$(OBJEXT) \ - gdb-dlfcn.$(OBJEXT) gdb_obstack.$(OBJEXT) \ - gdb_tilde_expand.$(OBJEXT) gdb_wait.$(OBJEXT) \ - gdb_vecs.$(OBJEXT) job-control.$(OBJEXT) netstuff.$(OBJEXT) \ - new-op.$(OBJEXT) pathstuff.$(OBJEXT) print-utils.$(OBJEXT) \ - ptid.$(OBJEXT) rsp-low.$(OBJEXT) run-time-clock.$(OBJEXT) \ - safe-strerror.$(OBJEXT) scoped_mmap.$(OBJEXT) search.$(OBJEXT) \ - signals.$(OBJEXT) signals-state-save-restore.$(OBJEXT) \ - tdesc.$(OBJEXT) thread-pool.$(OBJEXT) xml-utils.$(OBJEXT) \ - $(am__objects_1) + gdb-dlfcn.$(OBJEXT) gdb-hashtab.$(OBJEXT) \ + gdb_obstack.$(OBJEXT) gdb_tilde_expand.$(OBJEXT) \ + gdb_wait.$(OBJEXT) gdb_vecs.$(OBJEXT) job-control.$(OBJEXT) \ + netstuff.$(OBJEXT) new-op.$(OBJEXT) pathstuff.$(OBJEXT) \ + print-utils.$(OBJEXT) ptid.$(OBJEXT) rsp-low.$(OBJEXT) \ + run-time-clock.$(OBJEXT) safe-strerror.$(OBJEXT) \ + scoped_mmap.$(OBJEXT) search.$(OBJEXT) signals.$(OBJEXT) \ + signals-state-save-restore.$(OBJEXT) tdesc.$(OBJEXT) \ + thread-pool.$(OBJEXT) xml-utils.$(OBJEXT) $(am__objects_1) libgdbsupport_a_OBJECTS =3D $(am_libgdbsupport_a_OBJECTS) AM_V_P =3D $(am__v_P_@AM_V@) am__v_P_ =3D $(am__v_P_@AM_DEFAULT_V@) @@ -377,6 +377,7 @@ libgdbsupport_a_SOURCES =3D \ filestuff.cc \ format.cc \ gdb-dlfcn.cc \ + gdb-hashtab.cc \ gdb_obstack.cc \ gdb_tilde_expand.cc \ gdb_wait.cc \ @@ -483,6 +484,7 @@ distclean-compile: @AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/filestuff.Po@am__quote@ @AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/format.Po@am__quote@ @AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/gdb-dlfcn.Po@am__quote@ +@AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/gdb-hashtab.Po@am__quote@ @AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/gdb_obstack.Po@am__quote@ @AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/gdb_tilde_expand.Po@am__q= uote@ @AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/gdb_vecs.Po@am__quote@ diff --git a/gdbsupport/gdb-hashtab.cc b/gdbsupport/gdb-hashtab.cc new file mode 100644 index 00000000000..c42e8fec7b7 --- /dev/null +++ b/gdbsupport/gdb-hashtab.cc @@ -0,0 +1,44 @@ +/* Hash table wrappers for gdb. + Copyright (C) 2021 Free Software Foundation, Inc. + + This file is part of GDB. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . = */ + +#include "common-defs.h" +#include "gdb-hashtab.h" + +/* Allocation function for the libiberty hash table which uses an + obstack. The obstack is passed as DATA. */ + +void * +hashtab_obstack_allocate (void *data, size_t size, size_t count) +{ + size_t total =3D size * count; + void *ptr =3D obstack_alloc ((struct obstack *) data, total); + + memset (ptr, 0, total); + return ptr; +} + +/* Trivial deallocation function for the libiberty splay tree and hash + table - don't deallocate anything. Rely on later deletion of the + obstack. DATA will be the obstack, although it is not needed + here. */ + +void +dummy_obstack_deallocate (void *object, void *data) +{ + return; +} diff --git a/gdbsupport/gdb-hashtab.h b/gdbsupport/gdb-hashtab.h new file mode 100644 index 00000000000..65c2dc59a79 --- /dev/null +++ b/gdbsupport/gdb-hashtab.h @@ -0,0 +1,50 @@ +/* Hash table wrappers for gdb. + Copyright (C) 2021 Free Software Foundation, Inc. + + This file is part of GDB. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . = */ + +#ifndef GDBSUPPORT_GDB_HASHTAB_H +#define GDBSUPPORT_GDB_HASHTAB_H + +#include "hashtab.h" + +/* A deleter for a hash table. */ +struct htab_deleter +{ + void operator() (htab *ptr) const + { + htab_delete (ptr); + } +}; + +/* A unique_ptr wrapper for htab_t. */ +typedef std::unique_ptr htab_up; + +/* A wrapper for 'delete' that can used as a hash table entry deletion + function. */ +template +void +htab_delete_entry (void *ptr) +{ + delete (T *) ptr; +} + +/* Allocation and deallocation functions for the libiberty hash table + which use obstacks. */ +void *hashtab_obstack_allocate (void *data, size_t size, size_t count); +void dummy_obstack_deallocate (void *object, void *data); + +#endif /* GDBSUPPORT_GDB_HASHTAB_H */