[Lldb-commits] [PATCH] D12651: Add ctrl-c support to parallel dotest.py.

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Tue Sep 8 13:39:52 PDT 2015


Can you confirm with 2 runs before and 2 runs after the patch that you see
the same results every time?

Todd, can you think of a reason why this might happen?  I don't think it's
worth holding the patch up too long over this because I want to see the
other improvements come through, but if you can think of an obvious cause
for this Todd (or if you feel like looking into it before I get back
tomorrow Adrian) we could probably fix it before it goes in.  Either In any
case, I'm willing to let it go in this way and fix it tomorrow if necessary.

On Tue, Sep 8, 2015 at 1:29 PM Adrian McCarthy <amccarth at google.com> wrote:

> amccarth added a comment.
>
> Ninja check-lldb works.
>
> But...
>
> Before the patch, there are 53 failing test suites and 71 failing test
> cases.
> After the patch there are 54 failing test suites and 74 failing test cases.
>
> Diffing the list of failed suites at the end of the output shows only one
> difference:  TestCPPBreakpoints.py is listed twice after the patch.
>
> That may be unrelated to the patch, but it seems consistent.
>
>
> http://reviews.llvm.org/D12651
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150908/2ff60734/attachment.html>


More information about the lldb-commits mailing list