mbox series

[0/3] Handle dynamic properties and static link

Message ID 20230309-submit-static-link-fix-v1-0-23af27781fd2@adacore.com
Headers
Series Handle dynamic properties and static link |

Message

Tom Tromey March 9, 2023, 5:50 p.m. UTC 
  Ada uses static links and dynamic properties more than most other
languages.  A user found a bug when the two are combined, and this
series fixes the problem.

Regression tested on x86-64 Fedora 36.

Tom

---
Tom Tromey (3):
      Add frame parameter to resolve_dynamic_type
      Pass a frame to value_at_lazy and value_from_contents_and_address
      Use the correct frame when evaluating a dynamic property

 gdb/dwarf2/expr.c                          |  4 +-
 gdb/gdbtypes.c                             | 71 ++++++++++++++++++------------
 gdb/gdbtypes.h                             |  6 ++-
 gdb/testsuite/gdb.ada/static-link.exp      | 33 ++++++++++++++
 gdb/testsuite/gdb.ada/static-link/pck.ads  | 18 ++++++++
 gdb/testsuite/gdb.ada/static-link/prog.adb | 35 +++++++++++++++
 gdb/valops.c                               | 11 ++---
 gdb/value.c                                |  6 ++-
 gdb/value.h                                | 14 ++++--
 9 files changed, 156 insertions(+), 42 deletions(-)
---
base-commit: 093b56776fcf31ecc9342382341c1455fd397b16
change-id: 20230309-submit-static-link-fix-bcf714c48091

Best regards,

Comments

Tom Tromey March 29, 2023, 4:16 p.m. UTC | #1 
>>>>> "Tom" == Tom Tromey via Gdb-patches <gdb-patches@sourceware.org> writes:

Tom> Ada uses static links and dynamic properties more than most other
Tom> languages.  A user found a bug when the two are combined, and this
Tom> series fixes the problem.

Tom> Regression tested on x86-64 Fedora 36.

I'm checking this in now.

Tom