<div dir="ltr">Can you confirm with 2 runs before and 2 runs after the patch that you see the same results every time?<div><br></div><div>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.</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 8, 2015 at 1:29 PM Adrian McCarthy <<a href="mailto:amccarth@google.com">amccarth@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">amccarth added a comment.<br>
<br>
Ninja check-lldb works.<br>
<br>
But...<br>
<br>
Before the patch, there are 53 failing test suites and 71 failing test cases.<br>
After the patch there are 54 failing test suites and 74 failing test cases.<br>
<br>
Diffing the list of failed suites at the end of the output shows only one difference:  TestCPPBreakpoints.py is listed twice after the patch.<br>
<br>
That may be unrelated to the patch, but it seems consistent.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D12651" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12651</a><br>
<br>
<br>
<br>
</blockquote></div>