Re: [patch v9 08/23] gdbserver, linux, btrace: add btrace support for linux-low

Jan Kratochvil
2013-03-06T13:33:46Z

On Mon, 04 Mar 2013 18:05:55 +0100, Markus Metzger wrote:
> + if (tinfo != NULL)
> + tinfo->ptr_bits = register_size (0) * 8;

The diff against Pedro's review is OK; except for the Pedro's complaint on
PTR_BITS; but I see it is an abstraction as it is used in gdb/common/ while in
gdb/ one needs to use gdbarch_ptr_bit (gdbarch) and in gdbserver/ one needs to
use register_size (0).

There could be some more general abstraction put in gdb/common/ but I find it
acceptable as is for this patch.

Thanks,
Jan