<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Aug 23, 2014 at 4:07 PM, Dylan Noblesmith <span dir="ltr"><<a href="mailto:nobled@dreamwidth.org" target="_blank">nobled@dreamwidth.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":8qf" class="a3s" style="overflow:hidden">Support: add <span class="il">llvm</span>::unique_lock<br>
<br>
Based on the <span class="il">STL</span> class of the same name, it guards a <span class="il">mutex</span><br>
while also allowing it to be unlocked conditionally before<br>
destruction.</div></blockquote></div><br>I didn't notice this in the pre-commit review (the subject didn't really make it clear what you were planning on), but why do we need our own? Why can't we use std::unique_lock? I'm sure there is some reason, but I'm pretty sure it is a bug and not a feature and I'm somewhat opposed to us adding this kind of thing just to remove it later rather than fixing the underlying problems so that we can use std::unique_lock....</div>
</div>