This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 2/3] Set VALUE_VAL before set_value_address
On 11/22/2016 03:48 PM, Yao Qi wrote:
> Since we have a check on VALUE_VAL in set_value_address, we need to
> set VALUE_VAL properly before set_value_address.
LGTM.
I wonder whether it'd be hard to add a new function that
takes care of the ordering:
struct value *allocate_memory_value (struct type *type, CORE_ADDR address);
that would allocate a lazy value with lval == lval_memory and
address filled in. We use that instead throughout.
This would translate more directly to converting struct value to a
class hierarchy down the road, with allocate_memory_value mapping
to a struct memory_value constructor.
Thanks,
Pedro Alves