<div dir="ltr">I will revert this in a few hours and move forward with the changes we discussed offline, then resubmit.  Sorry!</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 16, 2014 at 7:51 AM, 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'm not really sure what's going on here, but I did not submit this, and I'm not sure why it has me listed as the submitter.  Maybe rnk@ did this without knowing what we discussed offline.</div>
<div class="HOEnZb"><div class="h5"><div class="gmail_extra">
<br><br><div class="gmail_quote">On Mon, Jun 16, 2014 at 2:23 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>I'm still really not happy with the approach for the global lock. This is<br>
completely different from the approach you and I discussed, on a separate<br>
patch with a separate review thread. Please *do not* fork reviews in this<br>
way. Please revert this and resume the code review on the original thread.<br>
=/ I'm sorry if you're frustrated by the time it is taking to get this in,<br>
but when one person has expressed specific concerns, I think it is a<br>
mistake to move forward without sync-ing back up with them.<br>
<br>
</div><a href="http://reviews.llvm.org/D4076" target="_blank">http://reviews.llvm.org/D4076</a><br>
<br>
<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>