bulbazord added inline comments.
================
Comment at: lldb/source/Target/DynamicRegisterInfo.cpp:204
+ m_sets.push_back(
+ {ConstString(set_name).AsCString(), nullptr, 0, nullptr});
} else {
----------------
mib wrote:
> I guess `m_sets` is a vector of `char*` ... Should we change it to
> `lldb::StringList` or `llvm::StringSet` so we don't have to create a
> `ConstString` here ?
`m_sets` is a vector of `lldb_private::RegisterSet` (which contains a `const
char *`). I think it's assumed that they are backed by a `ConstString`. Because
`RegisterSet` is defined in `lldb-private-types.h` we'd have to move it
somewhere else before we could actually use any types other than `const char
*`, I think?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152870/new/
https://reviews.llvm.org/D152870
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
