diff options
author | Joel Brobecker <brobecker@adacore.com> | 2013-11-07 08:35:35 +0400 |
---|---|---|
committer | Joel Brobecker <brobecker@adacore.com> | 2013-12-10 12:16:47 +0100 |
commit | 036e93dfdadd23ac73d3bd7f98f1640ded740cc0 (patch) | |
tree | 66624c6993134e63a1bd2db73a16142b52db2153 /gdb/testsuite | |
parent | 72bfa06c56ec662b4262cc264f45462ccda6a5d6 (diff) | |
download | gdb-036e93dfdadd23ac73d3bd7f98f1640ded740cc0.zip gdb-036e93dfdadd23ac73d3bd7f98f1640ded740cc0.tar.gz gdb-036e93dfdadd23ac73d3bd7f98f1640ded740cc0.tar.bz2 |
Set language for Ada minimal symbols.
This helps with the following issue: Given an Ada program defining
a global variable:
package Pck is
Watch : Integer := 1974;
end Pck;
When printing the address of this variable, GDB also tries to print
the associated symbol name:
(gdb) p watch'address
$1 = (access integer) 0x6139d8 <pck__watch>
^^
||
The problem is that GDB prints the variable's linkage name, instead
of its natural name. This is because the language of the associated
minimal symbol never really gets set.
This patch adds handling for Ada symbols in symbol_find_demangled_name.
After this patch, we now get:
(gdb) p watch'address
$1 = (access integer) 0x6139d8 <pck.watch>
^
|
gdb/ChangeLog:
* symtab.c (symbol_find_demangled_name): Add handling of
Ada symbols.
gdb/testsuite/ChangeLog:
* gdb.ada/int_deref.exp: Add test verifying that we print
the decoded symbol name when printing the address of Ada
symbols.
Diffstat (limited to 'gdb/testsuite')
-rw-r--r-- | gdb/testsuite/ChangeLog | 6 | ||||
-rw-r--r-- | gdb/testsuite/gdb.ada/int_deref.exp | 5 |
2 files changed, 11 insertions, 0 deletions
diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 08f6b27..0e1e785 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,5 +1,11 @@ 2013-12-10 Joel Brobecker <brobecker@adacore.com> + * gdb.ada/int_deref.exp: Add test verifying that we print + the decoded symbol name when printing the address of Ada + symbols. + +2013-12-10 Joel Brobecker <brobecker@adacore.com> + * gdb.mi/mi-start.exp: Add test verifying that -list-features contains "exec-run-start-option". diff --git a/gdb/testsuite/gdb.ada/int_deref.exp b/gdb/testsuite/gdb.ada/int_deref.exp index 7b8f426..ea61d4a 100644 --- a/gdb/testsuite/gdb.ada/int_deref.exp +++ b/gdb/testsuite/gdb.ada/int_deref.exp @@ -26,6 +26,11 @@ clean_restart ${testfile} set bp_location [gdb_get_line_number "Pck.Watch" ${testdir}/foo.adb] runto "foo.adb:$bp_location" +# Hijack a bit this testcase, to verify that name decoding works +# when doing symbolic address printing. +gdb_test "print watch'address" \ + " = \\(system\\.address\\) $hex <pck\\.watch>" + gdb_test "print *long_integer(watch'address)" \ " = 4874" |