[PATCH] D29588: Use rw locks for sanitizer thread registry

[Francis Ricci via Phabricator via llvm-commits]

fjricci added a comment.

Ok, I think that makes a lot of sense. I did originally think about something similar to the "proper solution" you mentioned, but I came to a similar conclusion about its complexity vs value.

In terms of the 2 solutions you proposed, the first seems quite a bit cleaner, and mirrors the behavior we already have on Linux. I'll go ahead and put up a diff for that, providing nobody has a strong preference for the second.


https://reviews.llvm.org/D29588