[Lldb-commits] [lldb] [lldb] Guard SBFrame/SBThread methods against running processes (PR #152020)

[Jonas Devlieghere via lldb-commits]

================
@@ -566,6 +559,53 @@ class ExecutionContext {
   lldb::StackFrameSP m_frame_sp; ///< The stack frame in thread.
 };
 
+/// A wrapper class representing an execution context with non-null Target
+/// and Process pointers, a locked API mutex and a locked ProcessRunLock.
+/// The locks are private by design: to unlock them, destroy the
+/// CompleteExecutionContext.
+struct CompleteExecutionContext : ExecutionContext {
----------------
JDevlieghere wrote:

If the goal is to discourage using `GetProcessPtr` and `GetTargetPtr`, I think we could mark them as deprecated for `CompleteExecutionContext` with a message that tells you to use the reference variants instead. 

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