<p dir="ltr"><br>
On Apr 4, 2013 4:52 AM, "Dmitry Vyukov" <<a href="mailto:dvyukov@google.com">dvyukov@google.com</a>> wrote:<br>
><br>
><br>
><br>
> ================<br>
> Comment at: lib/sanitizer_common/sanitizer_common.h:114<br>
> @@ -112,1 +113,3 @@<br>
> +// Can be used to prevent mixing error reports from different sanitizers.<br>
> +extern StaticSpinMutex CommonSanitizerReportMutex;<br>
><br>
> ----------------<br>
> We usually don't name variables LikeThis, we usually name them like_this.<br>
> And quite verbose on my taste. But feel free to ignore.<br>
><br>
> ================<br>
> Comment at: lib/tsan/rtl/tsan_rtl.h:525<br>
> @@ -524,3 +524,2 @@<br>
><br>
> -  Mutex report_mtx;<br>
>    int nreported;<br>
> ----------------<br>
> Leave this one as well. It detects deadlocks.<br>
> And it must the outer mutex, because it measures contention as well.</p>
<p dir="ltr">Must what?</p>
<p dir="ltr">><br>
><br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D618">http://llvm-reviews.chandlerc.com/D618</a><br>
</p>