aganea added a comment.
There's a typo in the title, `s/descirbes/describes/`
================
Comment at: lldb/source/Plugins/SymbolFile/NativePDB/PdbUtil.cpp:54
struct FindMembersSize : public TypeVisitorCallbacks {
- FindMembersSize(std::vector<std::pair<RegisterId, uint32_t>> &members_info,
+ FindMembersSize(llvm::SmallVector<std::pair<RegisterId, uint32_t>>
&members_info,
TpiStream &tpi)
----------------
labath wrote:
> I think `SmallVectorImpl` is still the official way to take SmallVector
> references.
`ArrayRef` would be even better! :) ie. `llvm::ArrayRef<std::pair<RegisterId,
uint32_t>> members_info;`
================
Comment at:
lldb/test/Shell/SymbolFile/NativePDB/subfield_register_simple_type.s:27
+# CHECK: LineEntry: [0x00401026-0x00401039): C:\src\a.cpp:3
+# CHECK-NEXT: Variable: id = {{.*}}, name = "x", type = "int64_t", valid
ranges = [0x0040102f-0x00401036), location = DW_OP_reg0 EAX, DW_OP_piece 0x4,
DW_OP_reg2 EDX, DW_OP_piece 0x4, decl =
+
----------------
nitpick: Is anything after (and including) ", valid ranges" necessary to cover
this change?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122943/new/
https://reviews.llvm.org/D122943
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
