================
@@ -936,3 +936,7 @@ size_t StackFrameList::GetStatus(Stream &strm, uint32_t
first_frame,
strm.IndentLess();
return num_frames_displayed;
}
+
+void StackFrameList::ClearSelectedFrameIndex() {
----------------
Michael137 wrote:
hmmmm good question. `GetSelectedFrameIndex` is not taking the mutex either. On
first glance I would've thought the mutex is there to just protect the
`m_frames`. There's a separate `m_inlined_depth_mutex` that protects the
corresponding member (for performance reason based on the comment next to it).
But if `ClearSelectedFrameIndex` and `GetSelectedFrameIndex` they should be
protected against concurrent access. Do we have concerns with using the list
mutex for that?
https://github.com/llvm/llvm-project/pull/133078
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
