I believe it does need to be a reclusive mutex. On Friday, July 16, 2021, Jonas Devlieghere via Phabricator < revi...@reviews.llvm.org> wrote:
> JDevlieghere accepted this revision. > JDevlieghere added inline comments. > This revision is now accepted and ready to land. > > > ================ > Comment at: lldb/include/lldb/Target/ThreadPlanStack.h:113 > std::unordered_map<size_t, PlanStack> m_completed_plan_store; > + mutable std::recursive_mutex m_stack_mutex; > }; > ---------------- > Does this actually have to be a recursive mutex? > > > Repository: > rG LLVM Github Monorepo > > CHANGES SINCE LAST ACTION > https://reviews.llvm.org/D106122/new/ > > https://reviews.llvm.org/D106122 > >
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
