public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Simon Marchi <simon.marchi@efficios.com>
To: gdb-patches@sourceware.org
Cc: Simon Marchi <simon.marchi@efficios.com>
Subject: [PATCH 00/24] C++ification of struct so_list
Date: Tue, 10 Oct 2023 16:39:55 -0400	[thread overview]
Message-ID: <20231010204213.111285-1-simon.marchi@efficios.com> (raw)

This series modernizes the struct so_list area by C++ifying it a little
bit and replacing the manual linked list implementation with
intrusive_list.  It also contains a few other cleanups written along the
way.

Simon Marchi (24):
  gdb: remove empty clear_solib functions
  gdb: add program_space parameter to target_so_ops::clear_solib
  gdb: make interps_notify work with references
  gdb: replace some so_list parameters to use references
  gdbsupport: use "reference" and "pointer" type aliases in
    intrusive_list
  gdbsupport: make intrusive_list's disposer accept a reference
  gdb: make get_cbfd_soname_build_id static
  gdb: allocate so_list with new, deallocate with delete
  gdb: rename lm_info_base to lm_info
  gdb: remove target_so_ops::free_so
  gdb: use gdb::checked_static_cast when casting lm_info
  gdb: make solib-svr4 not use so_list internally
  gdb: make solib-rocm not use so_list internally
  gdb: remove lm_info_vector typedef
  gdb: make so_list::lm_info a unique_ptr
  gdb: make clear_so a method of struct so_list
  gdb: remove target_section_table typedef
  gdb: make so_list::sections not a pointer
  gdb: make so_list::abfd a gdb_bfd_ref_ptr
  gdb: make so_list::{so_original_name,so_name} std::strings
  gdb: link so_list using intrusive_list
  gdb: don't call so_list::clear in free_so
  gdb: remove free_so function
  gdb: rename struct so_list to so

 gdb/bfd-target.c                         |   6 +-
 gdb/break-catch-load.c                   |   4 +-
 gdb/breakpoint.c                         |   6 +-
 gdb/bsd-uthread.c                        |  18 +-
 gdb/corelow.c                            |   7 +-
 gdb/exec.c                               |  24 +-
 gdb/exec.h                               |   6 +-
 gdb/hppa-tdep.c                          |   2 +-
 gdb/hppa-tdep.h                          |   4 +-
 gdb/inferior.c                           |  10 +-
 gdb/interps.c                            |  10 +-
 gdb/interps.h                            |  10 +-
 gdb/maint.c                              |   2 +-
 gdb/mi/mi-cmd-file.c                     |   6 +-
 gdb/mi/mi-interp.c                       |  26 +-
 gdb/mi/mi-interp.h                       |   6 +-
 gdb/nto-tdep.c                           |   6 +-
 gdb/nto-tdep.h                           |   3 +-
 gdb/observable.h                         |   7 +-
 gdb/progspace.c                          |   4 +-
 gdb/progspace.h                          |  26 +-
 gdb/record-full.c                        |   2 +-
 gdb/remote.c                             |   3 +-
 gdb/solib-aix.c                          |  69 +---
 gdb/solib-darwin.c                       |  84 ++---
 gdb/solib-dsbt.c                         |  73 ++--
 gdb/solib-frv.c                          |  58 ++--
 gdb/solib-rocm.c                         | 122 +++----
 gdb/solib-svr4.c                         | 421 ++++++++---------------
 gdb/solib-svr4.h                         |   4 +-
 gdb/solib-target.c                       | 121 +++----
 gdb/solib.c                              | 309 ++++++++---------
 gdb/solib.h                              |  13 +-
 gdb/solist.h                             |  77 ++---
 gdb/symfile.c                            |   2 +-
 gdb/symfile.h                            |   2 +-
 gdb/target-debug.h                       |   4 +-
 gdb/target-delegates.c                   |  14 +-
 gdb/target-section.h                     |   6 +-
 gdb/target.c                             |  10 +-
 gdb/target.h                             |   6 +-
 gdb/unittests/intrusive_list-selftests.c |   4 +-
 gdbsupport/intrusive_list.h              |  30 +-
 43 files changed, 658 insertions(+), 969 deletions(-)


base-commit: 635b2dd919b8c58e164b77c396041935fca1d66a
-- 
2.42.0


             reply	other threads:[~2023-10-10 20:42 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-10 20:39 Simon Marchi [this message]
2023-10-10 20:39 ` [PATCH 01/24] gdb: remove empty clear_solib functions Simon Marchi
2023-10-10 20:39 ` [PATCH 02/24] gdb: add program_space parameter to target_so_ops::clear_solib Simon Marchi
2023-10-17 14:57   ` Pedro Alves
2023-10-17 15:19     ` Simon Marchi
2023-10-10 20:39 ` [PATCH 03/24] gdb: make interps_notify work with references Simon Marchi
2023-10-11  8:48   ` Lancelot SIX
2023-10-11 14:18     ` Simon Marchi
2023-10-10 20:39 ` [PATCH 04/24] gdb: replace some so_list parameters to use references Simon Marchi
2023-10-19 11:07   ` [PATCH 4/24] " Lancelot SIX
2023-10-19 14:49     ` Simon Marchi
2023-10-19 15:20       ` Lancelot SIX
2023-10-19 16:07         ` Simon Marchi
2023-10-10 20:40 ` [PATCH 05/24] gdbsupport: use "reference" and "pointer" type aliases in intrusive_list Simon Marchi
2023-10-10 20:40 ` [PATCH 06/24] gdbsupport: make intrusive_list's disposer accept a reference Simon Marchi
2023-10-12 19:05   ` Pedro Alves
2023-10-14 20:12     ` Simon Marchi
2023-10-10 20:40 ` [PATCH 07/24] gdb: make get_cbfd_soname_build_id static Simon Marchi
2023-10-10 20:40 ` [PATCH 08/24] gdb: allocate so_list with new, deallocate with delete Simon Marchi
2023-10-10 20:40 ` [PATCH 09/24] gdb: rename lm_info_base to lm_info Simon Marchi
2023-10-10 20:40 ` [PATCH 10/24] gdb: remove target_so_ops::free_so Simon Marchi
2023-10-10 20:40 ` [PATCH 11/24] gdb: use gdb::checked_static_cast when casting lm_info Simon Marchi
2023-10-10 20:40 ` [PATCH 12/24] gdb: make solib-svr4 not use so_list internally Simon Marchi
2023-10-13 17:52   ` Lancelot SIX
2023-10-14 19:59     ` Simon Marchi
2023-10-19 11:08   ` Lancelot SIX
2023-10-19 14:50     ` Simon Marchi
2023-10-10 20:40 ` [PATCH 13/24] gdb: make solib-rocm " Simon Marchi
2023-10-13 18:35   ` Lancelot SIX
2023-10-14 20:00     ` Simon Marchi
2023-10-17 15:23   ` Pedro Alves
2023-10-17 15:32     ` Simon Marchi
2023-10-10 20:40 ` [PATCH 14/24] gdb: remove lm_info_vector typedef Simon Marchi
2023-10-10 20:40 ` [PATCH 15/24] gdb: make so_list::lm_info a unique_ptr Simon Marchi
2023-10-10 20:40 ` [PATCH 16/24] gdb: make clear_so a method of struct so_list Simon Marchi
2023-10-19 11:08   ` Lancelot SIX
2023-10-19 14:52     ` Simon Marchi
2023-10-10 20:40 ` [PATCH 17/24] gdb: remove target_section_table typedef Simon Marchi
2023-10-10 20:40 ` [PATCH 18/24] gdb: make so_list::sections not a pointer Simon Marchi
2023-10-10 20:40 ` [PATCH 19/24] gdb: make so_list::abfd a gdb_bfd_ref_ptr Simon Marchi
2023-10-10 20:40 ` [PATCH 20/24] gdb: make so_list::{so_original_name,so_name} std::strings Simon Marchi
2023-10-13 22:28   ` [PATCH 20/24] gdb: make so_list::{so_original_name, so_name} std::strings Lancelot SIX
2023-10-14 20:01     ` Simon Marchi
2023-10-19 11:08   ` [PATCH 20/24] gdb: make so_list::{so_original_name,so_name} std::strings Lancelot SIX
2023-10-19 14:55     ` Simon Marchi
2023-10-10 20:40 ` [PATCH 21/24] gdb: link so_list using intrusive_list Simon Marchi
2023-10-17 19:14   ` Pedro Alves
2023-10-17 19:38     ` Simon Marchi
2023-10-10 20:40 ` [PATCH 22/24] gdb: don't call so_list::clear in free_so Simon Marchi
2023-10-10 20:40 ` [PATCH 23/24] gdb: remove free_so function Simon Marchi
2023-10-10 20:49 ` [PATCH 24/24] gdb: rename struct so_list to so Simon Marchi
2023-10-17 19:20 ` [PATCH 00/24] C++ification of struct so_list Pedro Alves
2023-10-17 19:53   ` Simon Marchi
2023-10-20 14:40     ` Pedro Alves
2023-10-19 11:09 ` [PATCH 0/24] " Lancelot SIX
2023-10-19 14:57   ` Simon Marchi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20231010204213.111285-1-simon.marchi@efficios.com \
    --to=simon.marchi@efficios.com \
    --cc=gdb-patches@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).