[Lldb-commits] [PATCH] D77043: Use remote regnums in expedited list, value regs and invalidate regs
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Jan 14 01:08:59 PST 2021
labath added a comment.
Using remote numbers in the lists seems reasonable as well, but I am troubled by the "opportunistic" aspect of the implementation. This pattern
uint32_t lldb_regnum = ConvertRegisterKindToRegisterNumber(
eRegisterKindProcessPlugin, reg);
if (lldb_regnum != LLDB_INVALID_REGNUM)
reg = lldb_regnum;
const RegisterInfo *value_reg_info = GetRegisterInfoAtIndex(reg);
is cumbersome to write and easy to get wrong.
Why are we sometimes **not** using the plugin/remote number? Could we make it so that this number is used unconditionally?
Then all of this could become: `const RegisterInfo *value_reg_info = GetRegisterInfo(eRegisterKindProcessPlugin, reg)`
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77043/new/
https://reviews.llvm.org/D77043
More information about the lldb-commits
mailing list