[Lldb-commits] [PATCH] D13124: test runner: switch to pure-Python timeout mechanism

Todd Fiala via lldb-commits lldb-commits at lists.llvm.org
Tue Sep 29 12:58:53 PDT 2015


Great, thanks!  Will do after I fix up your bits.  Out for a bit but will
get this in the afternoon.

On Tue, Sep 29, 2015 at 12:53 PM, Zachary Turner <zturner at google.com> wrote:

> Unless you make significant other changes, feel free to just commit.  I
> don't have any other concerns.  Thanks for working on this!
>
> On Tue, Sep 29, 2015 at 12:51 PM Todd Fiala <todd.fiala at gmail.com> wrote:
>
>> tfiala added a comment.
>>
>> > Does that look right to you?
>>
>>
>> That result looks right - the two soft terminate tests get skipped when
>> it's not reported.
>>
>> If you ran the lldb test suite with that setup, they should work as well
>> as they did before the change.  (If they don't, I'm missing a lower level
>> test).
>>
>> I'll fix up those other parts you called out and get what is hopefully
>> the final patch up.
>>
>>
>> http://reviews.llvm.org/D13124
>>
>>
>>
>>


-- 
-Todd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150929/018ace7a/attachment-0001.html>


More information about the lldb-commits mailing list