[PATCH] Remove support for enabling / disabling multi-threading at runtime.

Zachary Turner zturner at google.com
Mon Jun 16 07:52:49 PDT 2014


I will revert this in a few hours and move forward with the changes we
discussed offline, then resubmit.  Sorry!


On Mon, Jun 16, 2014 at 7:51 AM, Zachary Turner <zturner at google.com> wrote:

> 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.
>
>
> On Mon, Jun 16, 2014 at 2:23 AM, Chandler Carruth <chandlerc at gmail.com>
> wrote:
>
>> I'm still really not happy with the approach for the global lock. This is
>> completely different from the approach you and I discussed, on a separate
>> patch with a separate review thread. Please *do not* fork reviews in this
>> way. Please revert this and resume the code review on the original thread.
>> =/ I'm sorry if you're frustrated by the time it is taking to get this in,
>> but when one person has expressed specific concerns, I think it is a
>> mistake to move forward without sync-ing back up with them.
>>
>> http://reviews.llvm.org/D4076
>>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140616/5d595086/attachment.html>


More information about the llvm-commits mailing list