[Lldb-commits] [PATCH] D116461: [lldb] Remove ProcessStructReader from NSStringSummaryProvider (NFC)

Dave Lee via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Sat Jan 8 09:27:58 PST 2022


kastiglione added a comment.

To add more context, `ProcessStructReader` is used in only a few places, and none seem like they should truly need it. This was a low hanging fruit.

Something I don't understand about this code is why it the objc metadata is incomplete. The class is known, but not its ivars, why? In theory this code should be able to request the `_lengthAndRefCount` child member directly by name.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116461/new/

https://reviews.llvm.org/D116461



More information about the lldb-commits mailing list