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

[via lldb-commits]

================
@@ -279,7 +282,11 @@ class ExecutionContextRef {
                                               ///< object refers to in case the
                                               /// backing object changes
   StackID m_stack_id; ///< The stack ID that this object refers to in case the
-                      ///backing object changes
+                      ///< backing object changes
----------------
jimingham wrote:

I'd say something stronger than "used to avoid circular dependencies".  If this WP resolves to a StackFrameListSP, then this is the list you MUST use to resolve StackID.  If it isn't set, then you should ask the Thread to decide what StackFrame that StackID resolves to.

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