[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