Re: [PATCH 8/9] gdb: remove VALUE_REGNUM, add value::regnum

Simon Marchi
2023-12-22T16:58:11Z

On 2023-12-22 11:52, Tom Tromey wrote:
>>>>>> "Simon" == Simon Marchi <simon.marchi@efficios.com> writes:
>
> Simon> +/* See value.h. */
> Simon> +
> Simon> +int
> Simon> +value::regnum ()
> Simon> {
> Simon> gdb_assert (m_lval == lval_register);
> Simon> - return &m_location.reg.regnum;
> Simon> +
> Simon> + return m_location.reg.regnum;
>
> I wonder if this could just be inlined in the header.
>
> Tom

Yes, I think it's a good candidate to be in the header, I'll do that.

Simon