From: Tom Tromey <tromey@adacore.com>
To: gdb-patches@sourceware.org
Cc: Tom Tromey <tromey@adacore.com>
Subject: [PATCH 0/3] Add compatibility definitions for C++ thread support
Date: Wed, 01 Oct 2025 12:57:46 -0600 [thread overview]
Message-ID: <20251001-isolate-cxx-thread-v1-0-4bb740be2467@adacore.com> (raw)
This series adds some C++ compatibility support code, with the goal of
removing most checks of CXX_STD_THREAD -- while it's not really
practical to remove all the checks from gdb, the majority can be
removed using this technique. This makes the code somewhat cleaner,
at the expense of having to remember to use the "gdb" namespace rather
than "std". Perhaps some day we could remove the shims as well.
Regression tested on x86-64 Fedora 41, both with an ordinary build and
with a --disable-threading build. The latter has one pre-existing
failure.
Signed-off-by: Tom Tromey <tromey@adacore.com>
---
Tom Tromey (3):
Remove an unused include
Don't rely on deduction in run-on-main-thread.c
Introduce gdbsupport/cxx-thread.h and use it
gdb/complaints.c | 16 +--
gdb/dwarf2/cooked-index-worker.c | 17 +--
gdb/dwarf2/cooked-index-worker.h | 17 +--
gdb/dwarf2/read.c | 36 ++----
gdb/dwarf2/read.h | 8 +-
gdb/gdb_bfd.c | 47 ++------
gdb/minsyms.c | 37 ++----
gdb/run-on-main-thread.c | 38 ++----
gdbsupport/cxx-thread.h | 243 +++++++++++++++++++++++++++++++++++++++
gdbsupport/thread-pool.h | 91 +--------------
10 files changed, 297 insertions(+), 253 deletions(-)
---
base-commit: 52ca3d3fe6139ec45e04b4cfb1b5e244a8459586
change-id: 20251001-isolate-cxx-thread-ee03d199ab5b
Best regards,
--
Tom Tromey <tromey@adacore.com>
next reply other threads:[~2025-10-01 18:59 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-01 18:57 Tom Tromey [this message]
2025-10-01 18:57 ` [PATCH 1/3] Remove an unused include Tom Tromey
2025-10-01 20:04 ` Simon Marchi
2025-10-02 12:59 ` Tom Tromey
2025-10-01 18:57 ` [PATCH 2/3] Don't rely on deduction in run-on-main-thread.c Tom Tromey
2025-10-01 18:57 ` [PATCH 3/3] Introduce gdbsupport/cxx-thread.h and use it Tom Tromey
2025-10-01 20:16 ` Simon Marchi
2025-10-02 16:58 ` Tom Tromey
2025-10-02 18:09 ` Tom Tromey
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=20251001-isolate-cxx-thread-v1-0-4bb740be2467@adacore.com \
--to=tromey@adacore.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