JDevlieghere added a comment. I think you might be looking at a combination of the old and the new patch. The new mutex protects the whole `Cancel` and `SetIsRunning`. I don't think this needs to be a recursive mutex because these functions are not calling each other. They both indirectly protect access to `SetIsDone`.
CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120762/new/ https://reviews.llvm.org/D120762 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits