[Lldb-commits] [lldb] [lldb/Target] Track containing StackFrameList to avoid circular dependencies (PR #170226)

via lldb-commits lldb-commits at lists.llvm.org
Mon Dec 1 16:56:23 PST 2025


https://github.com/jimingham approved this pull request.

This LGTM except for a suggested change in the comment for m_frame_list_wp.  You make it sound like this is some optional thing that you could choose to use or not.  But if it is resolveable it IS the stack frame list in which you are supposed to look up m_stack_id.  Your comment needs to say that.
Approved provided the comment is fixed.

https://github.com/llvm/llvm-project/pull/170226


More information about the lldb-commits mailing list