[lldb-dev] [PATCH] make dosep.ty test driver multi-threadable

[Todd Fiala]

I'd add this diff to it to clean up output that is now sometimes incorrect:

@@ -56,7 +57,7 @@ def walk_and_invoke(test_root, dotest_options,
num_threads):
     failed = []
     passed = []
     if (num_threads > 1):
-        print "Running multithreaded with " + str(num_threads) + " threads
(from " + multithreaded_envar + ")"
+        print "Running multithreaded with " + str(num_threads) + "
threads."
         global in_q
         global out_q
         in_q = Queue.Queue()



On Mon, Mar 17, 2014 at 2:29 PM, Todd Fiala <tfiala at google.com> wrote:

> (and ran the right number of threads, which for me is 32).
>
>
> On Mon, Mar 17, 2014 at 2:29 PM, Todd Fiala <tfiala at google.com> wrote:
>
>> This patch worked for me.
>>
>>
>> On Mon, Mar 17, 2014 at 2:25 PM, Todd Fiala <tfiala at google.com> wrote:
>>
>>> ah ok.
>>>
>>>
>>> On Mon, Mar 17, 2014 at 2:25 PM, Ed Maste <emaste at freebsd.org> wrote:
>>>
>>>> > I don't know that it's entirely clear what the best way is to pass
>>>> thread arguments to dosep.ty via either of those.  Do we want to add
>>>> --threads to the default instantiations of both of those so it picks up a
>>>> good default?
>>>>
>>>> Greg's change defaults to # of CPUs if not explicitly set, so no need
>>>> to pass a new argument.
>>>>
>>>
>>>
>>>
>>> --
>>> Todd Fiala | Software Engineer |  tfiala at google.com |  650-943-3180
>>>
>>
>>
>>
>> --
>> Todd Fiala | Software Engineer |  tfiala at google.com |  650-943-3180
>>
>
>
>
> --
> Todd Fiala | Software Engineer |  tfiala at google.com |  650-943-3180
>



-- 
Todd Fiala | Software Engineer | tfiala at google.com | 650-943-3180
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20140317/7139693b/attachment.html>