Add new overload of gdbarch_return_value
authorTom Tromey <tromey@adacore.com>
Wed, 7 Sep 2022 14:39:52 +0000 (08:39 -0600)
committerTom Tromey <tromey@adacore.com>
Tue, 3 Jan 2023 15:45:00 +0000 (08:45 -0700)
commit4e1d2f5814b29048d1dd1cea2cb50570e6c8f1f8
treec5b684d3cad8e726247e58566b9d293ece12b756
parent862ebb27bbe30768356776a10827dd1c0824d405
Add new overload of gdbarch_return_value

The gdbarch "return_value" can't correctly handle variably-sized
types.  The problem here is that the TYPE_LENGTH of such a type is 0,
until the type is resolved, which requires reading memory.  However,
gdbarch_return_value only accepts a buffer as an out parameter.

Fixing this requires letting the implementation of the gdbarch method
resolve the type and return a value -- that is, both the contents and
the new type.

After an attempt at this, I realized I wouldn't be able to correctly
update all implementations (there are ~80) of this method.  So,
instead, this patch adds a new method that falls back to the current
method, and it updates gdb to only call the new method.  This way it's
possible to incrementally convert the architectures that I am able to
test.
gdb/arch-utils.c
gdb/arch-utils.h
gdb/elfread.c
gdb/gdbarch-components.py
gdb/gdbarch-gen.h
gdb/gdbarch.c
gdb/infcall.c
gdb/infcmd.c
gdb/stack.c
gdb/value.c