diff options
| author | Tom Tromey <tom@tromey.com> | 2024-12-18 18:49:45 -0700 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2025-01-05 11:58:49 -0700 |
| commit | 64800e8954245cd32c797fb2e207a66793193bf3 (patch) | |
| tree | b3e8232341ffb9455d52568a6f191c76310ae14e /gdb/testsuite/gdb.python/py-recurse-unwind.py | |
| parent | 05248f5ff0b92ada207dea4fab8f20df35746564 (diff) | |
| download | binutils-64800e8954245cd32c797fb2e207a66793193bf3.zip binutils-64800e8954245cd32c797fb2e207a66793193bf3.tar.gz binutils-64800e8954245cd32c797fb2e207a66793193bf3.tar.bz2 | |
Fix latent bug in Ada import symbol handling
The code in dwarf2/read.c:new_symbol that handles Ada 'import' symbols
has a bug. It uses the current scope, which by default this is the
file scope -- even for a global symbol like:
<1><1186>: Abbrev Number: 4 (DW_TAG_variable)
<1187> DW_AT_name : (indirect string, offset: 0x1ad2): pkg__imported_var_ada
...
<1196> DW_AT_external : 1
This disagrees with the scope computed by the DWARF indexer.
Now, IMO new_symbol and its various weirdness really has to go. And,
ideally, this information would come from the indexer rather than
perhaps being erroneously recomputed. But meanwhile, this patch fixes
the issue at hand.
This came up while working on another change that exposes the bug.
Reviewed-By: Tom de Vries <tdevries@suse.de>
Diffstat (limited to 'gdb/testsuite/gdb.python/py-recurse-unwind.py')
0 files changed, 0 insertions, 0 deletions