I believe it does need to be a reclusive mutex.<br><br>On Friday, July 16, 2021, Jonas Devlieghere via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">JDevlieghere accepted this revision.<br>
JDevlieghere added inline comments.<br>
This revision is now accepted and ready to land.<br>
<br>
<br>
================<br>
Comment at: lldb/include/lldb/Target/<wbr>ThreadPlanStack.h:113<br>
   std::unordered_map<size_t, PlanStack> m_completed_plan_store;<br>
+  mutable std::recursive_mutex m_stack_mutex;<br>
 };<br>
----------------<br>
Does this actually have to be a recursive mutex? <br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D106122/new/" target="_blank">https://reviews.llvm.org/<wbr>D106122/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D106122" target="_blank">https://reviews.llvm.org/<wbr>D106122</a><br>
<br>
</blockquote>