public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Tom Tromey <tromey@adacore.com>
To: gdb-patches@sourceware.org
Subject: [PATCH v2 0/8] Implement Ada Pragma Import and Pragma Export
Date: Fri, 07 Apr 2023 10:39:53 -0600	[thread overview]
Message-ID: <20230314-submit-pragma-import-export-v2-0-dcff927191fb@adacore.com> (raw)

This adds better support to GDB for Ada's import and export feature.
There are some minor cleanups to start with, then the bulk of the
feature in the last patch, which I think should be reasonably
self-explanatory.

Regression tested on x86-64 Fedora 36.  It's also been running in the
AdaCore internal test suite for a while.

Tom

---
Changes in v2:
- Adds fix for a regression found internally at AdaCore.
- Link to v1: https://inbox.sourceware.org/gdb-patches/20230314-submit-pragma-import-export-v1-0-a235709f7e96@adacore.com

---
Tom Tromey (8):
      Introduce lookup_minimal_symbol_linkage
      Bump MAX_SYMBOL_IMPLS
      Define symbol::value_block separately
      Introduce symbol_block_ops::get_block_value
      Handle Ada Pragma Import and Pragma Export
      Use reference parameter in remove_extra_symbols
      Use bool and early loop exit in remove_extra_symbols
      Fix regression due to Pragma Import series

 gdb/Makefile.in                          |   1 +
 gdb/ada-lang.c                           |  70 ++++++++++-------
 gdb/dwarf2/ada-imported.c                | 122 +++++++++++++++++++++++++++++
 gdb/dwarf2/loc.h                         |   2 +
 gdb/dwarf2/read.c                        | 128 +++++++++++++++++++++++++++++++
 gdb/minsyms.c                            |  22 ++++++
 gdb/minsyms.h                            |   8 ++
 gdb/symtab.c                             |  36 +++------
 gdb/symtab.h                             |  20 ++++-
 gdb/testsuite/gdb.ada/complete.exp       |   2 +
 gdb/testsuite/gdb.ada/complete/pck.ads   |   4 +-
 gdb/testsuite/gdb.ada/import.exp         |  60 +++++++++++++++
 gdb/testsuite/gdb.ada/import/inc.c       |  27 +++++++
 gdb/testsuite/gdb.ada/import/pkg.adb     |  35 +++++++++
 gdb/testsuite/gdb.ada/import/pkg.ads     |  38 +++++++++
 gdb/testsuite/gdb.ada/import/prog.adb    |  31 ++++++++
 gdb/testsuite/gdb.ada/info_auto_lang.exp |  10 ++-
 17 files changed, 555 insertions(+), 61 deletions(-)
---
base-commit: 9d1c303d52d7aeb2ad0217e53e2bf0a840b93316
change-id: 20230314-submit-pragma-import-export-c38bb4077953

Best regards,
-- 
Tom Tromey <tromey@adacore.com>


             reply	other threads:[~2023-04-07 16:39 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-07 16:39 Tom Tromey [this message]
2023-04-07 16:39 ` [PATCH v2 1/8] Introduce lookup_minimal_symbol_linkage Tom Tromey
2023-04-07 16:39 ` [PATCH v2 2/8] Bump MAX_SYMBOL_IMPLS Tom Tromey
2023-04-07 16:39 ` [PATCH v2 3/8] Define symbol::value_block separately Tom Tromey
2023-04-07 16:39 ` [PATCH v2 4/8] Introduce symbol_block_ops::get_block_value Tom Tromey
2023-04-07 16:39 ` [PATCH v2 5/8] Handle Ada Pragma Import and Pragma Export Tom Tromey
2023-04-07 16:39 ` [PATCH v2 6/8] Use reference parameter in remove_extra_symbols Tom Tromey
2023-04-07 16:40 ` [PATCH v2 7/8] Use bool and early loop exit " Tom Tromey
2023-04-07 16:40 ` [PATCH v2 8/8] Fix regression due to Pragma Import series Tom Tromey
2023-05-12 19:25 ` [PATCH v2 0/8] Implement Ada Pragma Import and Pragma Export 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=20230314-submit-pragma-import-export-v2-0-dcff927191fb@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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).