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

Reply via email to