JDevlieghere added a comment.

In D157654#4581578 <https://reviews.llvm.org/D157654#4581578>, @augusto2112 
wrote:

> @JDevlieghere I updated this to two shared mutexes because I'm assuming you 
> can have more than one concurrent read and more than one concurrent write. If 
> this is wrong I can go back to two regular mutexes.

According to POSIX [1] that would be undefined:

> The behavior of multiple concurrent reads on the same pipe, FIFO, or terminal 
> device is unspecified.

A regular mutex should be fine.

[1] https://pubs.opengroup.org/onlinepubs/9699919799/functions/read.html


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157654/new/

https://reviews.llvm.org/D157654

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to