<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 13, 2014 at 11:01 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I tried this once and ran into some difficulty, because before this change, the Mutex is a pointer which null is an expected value if llvm_start_multithreaded has not been called.  So the function would have to return a pointer to the mutex.  That would require changing MutexGuard to take a pointer instead of a reference, which would generate a ton of churn.<div>

<br></div><div>I'll have another stab at it and see if there's a way to do it in a way that has very little churn and still contributes to this change as well.</div></div></blockquote></div><br>I would make there always be a mutex as you do in this change? Is there some reason that dosen't work? I'm not worried about these being no-ops when multithreading is disabled as long as they're correct.</div>
</div>