diff options
| author | Tom Tromey <tromey@adacore.com> | 2021-04-15 10:05:00 -0600 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2021-04-15 10:05:00 -0600 |
| commit | 9e5e03df52968b416e09a59482409abfed9727c0 (patch) | |
| tree | f7b84a83f5c0aa2f636927ef25efed33604c3f13 /gdb/parse.c | |
| parent | 4c79248a46100016e902f1489ce1c38a42608460 (diff) | |
| download | gdb-9e5e03df52968b416e09a59482409abfed9727c0.zip gdb-9e5e03df52968b416e09a59482409abfed9727c0.tar.gz gdb-9e5e03df52968b416e09a59482409abfed9727c0.tar.bz2 | |
Use block_symbol in var_value_operation
I noticed that var_value_operation takes a block and a symbol, and
most callers destructure a block_symbol to pass in. It seems better
for this class to simply hold a block_symbol instead.
Tested on x86-64 Fedora 32.
gdb/ChangeLog
2021-04-15 Tom Tromey <tromey@adacore.com>
* rust-exp.y (rust_parser::convert_ast_to_expression): Update.
* parse.c (parser_state::push_symbol, parser_state::push_dollar):
Update.
* p-exp.y (variable): Update.
* m2-exp.y (variable): Update.
* go-exp.y (variable): Update.
* expprint.c (dump_for_expression): New overload.
* expop.h (check_objfile): New overload.
(check_constant): New overload.
(class var_value_operation): Use block_symbol.
<get_symbol>: Rewrite.
* eval.c (var_value_operation::evaluate)
(var_value_operation::evaluate_funcall)
(var_value_operation::evaluate_for_address)
(var_value_operation::evaluate_for_address)
(var_value_operation::evaluate_with_coercion)
(var_value_operation::evaluate_for_sizeof)
(var_value_operation::evaluate_for_cast): Update.
* d-exp.y (PrimaryExpression): Update.
* c-exp.y (variable): Update.
* ax-gdb.c (var_value_operation::do_generate_ax): Update.
* ada-lang.c (ada_var_value_operation::evaluate_for_cast)
(ada_var_value_operation::evaluate)
(ada_var_value_operation::resolve)
(ada_funcall_operation::resolve): Update.
* ada-exp.y (write_var_from_sym, write_object_renaming)
(write_ambiguous_var, write_var_or_type, write_name_assoc)
(maybe_overload): Update.
* ada-exp.h (class ada_var_value_operation) <get_block>: Rewrite.
Diffstat (limited to 'gdb/parse.c')
| -rw-r--r-- | gdb/parse.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gdb/parse.c b/gdb/parse.c index 692d38d..b2f23eb 100644 --- a/gdb/parse.c +++ b/gdb/parse.c @@ -224,7 +224,7 @@ parser_state::push_symbol (const char *name, block_symbol sym) { if (symbol_read_needs_frame (sym.symbol)) block_tracker->update (sym); - push_new<expr::var_value_operation> (sym.symbol, sym.block); + push_new<expr::var_value_operation> (sym); } else { @@ -301,7 +301,7 @@ parser_state::push_dollar (struct stoken str) sym = lookup_symbol (copy.c_str (), NULL, VAR_DOMAIN, NULL); if (sym.symbol) { - push_new<expr::var_value_operation> (sym.symbol, sym.block); + push_new<expr::var_value_operation> (sym); return; } msym = lookup_bound_minimal_symbol (copy.c_str ()); |