diff options
author | Zoran Zaric <zoran.zaric@amd.com> | 2020-09-15 12:35:56 +0100 |
---|---|---|
committer | Zoran Zaric <zoran.zaric@amd.com> | 2021-08-05 16:40:26 +0100 |
commit | f9e4ed8baa9eeebc71be88f863c52f81e42bed34 (patch) | |
tree | fae4b1fd02894866edb1fd00402028afb5b7ca4b /gdb/dwarf2/expr.h | |
parent | 14a62404c9845827d92986af2dfcdbb7e48c65fc (diff) | |
download | binutils-f9e4ed8baa9eeebc71be88f863c52f81e42bed34.zip binutils-f9e4ed8baa9eeebc71be88f863c52f81e42bed34.tar.gz binutils-f9e4ed8baa9eeebc71be88f863c52f81e42bed34.tar.bz2 |
Merge evaluate_for_locexpr_baton evaluator
The evaluate_for_locexpr_baton is the last derived class from the
dwarf_expr_context class. It's purpose is to support the passed in
buffer functionality.
Although, it is not really necessary to merge this class with it's
base class, doing that simplifies new expression evaluator design.
Considering that this functionality is going around the DWARF standard,
it is also reasonable to expect that with a new evaluator design and
extending the push object address functionality to accept any location
description, there will be no need to support passed in buffers.
Alternatively, it would also makes sense to abstract the interaction
between the evaluator and a given resource in the near future. The
passed in buffer would then be a specialization of that abstraction.
gdb/ChangeLog:
* dwarf2/expr.c (dwarf_expr_context::read_mem): Merge with
evaluate_for_locexpr_baton implementation.
* dwarf2/loc.c (class evaluate_for_locexpr_baton): Remove
class.
(evaluate_for_locexpr_baton::read_mem): Move to
dwarf_expr_context.
(dwarf2_locexpr_baton_eval): Instantiate dwarf_expr_context
instead of evaluate_for_locexpr_baton class.
Diffstat (limited to 'gdb/dwarf2/expr.h')
-rw-r--r-- | gdb/dwarf2/expr.h | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/gdb/dwarf2/expr.h b/gdb/dwarf2/expr.h index 9dc9257..76c073c 100644 --- a/gdb/dwarf2/expr.h +++ b/gdb/dwarf2/expr.h @@ -191,16 +191,8 @@ struct dwarf_expr_context /* Object address used for the evaluation. */ CORE_ADDR obj_address = 0; - /* Read LENGTH bytes at ADDR into BUF. */ - virtual void read_mem (gdb_byte *buf, CORE_ADDR addr, size_t length); - - /* Return the `object address' for DW_OP_push_object_address. */ - virtual CORE_ADDR get_object_address () - { - if (obj_address == 0) - error (_("Location address is not set.")); - return obj_address; - } + /* The data that was passed in. */ + gdb::array_view<const gdb_byte> data_view; private: @@ -234,6 +226,12 @@ private: void push_dwarf_reg_entry_value (call_site_parameter_kind kind, call_site_parameter_u kind_u, int deref_size); + + /* Read LENGTH bytes at ADDR into BUF. This method also handles the + case where a caller of the evaluator passes in some data, + but with the address being 0. In this situation, we arrange for + memory reads to come from the passed-in buffer. */ + void read_mem (gdb_byte *buf, CORE_ADDR addr, size_t length); }; /* Return the value of register number REG (a DWARF register number), |