diff options
author | Joel Brobecker <brobecker@gnat.com> | 2012-03-02 19:29:01 +0000 |
---|---|---|
committer | Joel Brobecker <brobecker@gnat.com> | 2012-03-02 19:29:01 +0000 |
commit | a5ee536be2c16fcbe223490358dbe4d1914f0e1a (patch) | |
tree | f12b46a7e65a78cd75c0ea488ab154ea25e6d765 /gdb/d-lang.c | |
parent | f59f708a4b12a347cb003c750b96336822481900 (diff) | |
download | binutils-a5ee536be2c16fcbe223490358dbe4d1914f0e1a.zip binutils-a5ee536be2c16fcbe223490358dbe4d1914f0e1a.tar.gz binutils-a5ee536be2c16fcbe223490358dbe4d1914f0e1a.tar.bz2 |
language-specific read_var_value for Ada renamings
The purpose of this patch is to better support renamings in the
"info locals" command. Consider ...
procedure Foo is
GV : Integer renames Pck.Global_Variable;
begin
Increment (GV); -- STOP
end Foo;
... Pck.Global_Variable is just an integer. After having stopped at
the "STOP" line, "info locals" yields:
(gdb) info locals
gv = <error reading variable gv (Cannot access memory at address 0xffffffffffffffff)>
In reality, two things are happening:
(1) Variable "GV" does not exist, which is normal, since there is
"GV" the renaming of another variable;
(2) But to allow the user access to that renaming the same way
the code has, the compiler produces an artificial variable
whose name encodes the renaming:
gv___XR_pck__global_variable___XE
For practical reasons, the artificial variable itself is given
irrelevant types and addresses.
But the "info locals" command does not act as if it was a short-cut
of "foreach VAR in locals, print VAR". Instead it gets the value of
each VAR directly, which does not work in this case, since the variable
is artificial and needs to be decoded first.
This patch makes the "read_var_value" routine language-specific.
The old implementation of "read_var_value" gets renamed to
"default_read_var_value" and all languages now use it (unchanged
behavior), except for Ada. In Ada, the new function ada_read_var_value
checks if we have a renaming, and if so, evaluates its value, or else
defers to default_read_var_value.
gdb/ChangeLog:
* language.h (struct language_defn): New "method" la_read_var_value.
* findvar.c: #include "language.h".
(default_read_var_value): Renames read_var_value. Rewrite
function description.
(read_var_value): New function.
* value.h (default_read_var_value): Add prototype.
* ada-lang.c (ada_read_renaming_var_value, ada_read_var_value):
New functions.
(ada_language_defn): Add entry for la_read_var_value.
* c-lang.c, d-lang.c, f-lang.c, jv-lang.c, language.c,
* m2-lang.c, objc-lang.c, opencl-lang.c, p-lang.c: Update
language_defn structures to add entry for new la_read_var_value
field.
Diffstat (limited to 'gdb/d-lang.c')
-rw-r--r-- | gdb/d-lang.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/gdb/d-lang.c b/gdb/d-lang.c index 202820e..9a8cf52 100644 --- a/gdb/d-lang.c +++ b/gdb/d-lang.c @@ -256,6 +256,7 @@ static const struct language_defn d_language_defn = syntax. */ d_val_print, /* Print a value using appropriate syntax. */ c_value_print, /* Print a top-level value. */ + default_read_var_value, /* la_read_var_value */ NULL, /* Language specific skip_trampoline. */ "this", basic_lookup_symbol_nonlocal, |