<p dir="ltr">I do not have commit access so if someone who does could commit that would be great.</p>
<p dir="ltr">Thanks,<br>
Scott Knight</p>
<div class="gmail_quote">On May 16, 2014 3:26 PM,  <<a href="mailto:jingham@apple.com">jingham@apple.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I got the same result.  This patch is much more likely to cause crashes than failures if it was wrong, and I didn't see that either.  And the patch looks reasonable.  I say check it in.<br>
<br>
Jim<br>
<br>
On May 16, 2014, at 12:23 PM, Scott Knight <<a href="mailto:knightsc@gmail.com">knightsc@gmail.com</a>> wrote:<br>
<br>
> I did go ahead and run dotest.py this morning both before and after applying the patch and it seems to have the same 10 failures before and after like you mentioned. Another set of eyes checking the test results would be great though.<br>

><br>
> <a href="http://reviews.llvm.org/D3616" target="_blank">http://reviews.llvm.org/D3616</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br>
</blockquote></div>