================
@@ -470,6 +470,18 @@ class Thread : public std::enable_shared_from_this<Thread>,
virtual void ClearStackFrames();
+ /// Derived classes implementing SetBackingThread should use this to provide
+ /// bidirectional access to the Backing-Backed relationship.
+ void SetBackedThread(Thread &backed_thread) {
----------------
jimingham wrote:
You have an assert that's requiring that `backed_thread` has to have had its
backing thread set first, then passed to this API. Would it be a more
user-friendly API if it set the backed_thread's backing thread here, if it
isn't already set.
It's still fine to assert if somebody set the wrong backing thread, that seems
like an unintended error. But if the backed_thread.GetBackingThread was null,
is there any reason not to set it for them here?
https://github.com/llvm/llvm-project/pull/125300
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
