<div dir="ltr">If the patch is busted, pretty much every test should start failing.  As long as ninja check-lldb actually runs, completes, and behaves pretty much the same way as before, that's pretty much all that needs to be verified.<div><br></div><div>Ctrl+C shouldn't behave any differently with this patch, the idea was to refactor it so that on Windows it would be identical to pre-patch, but on other platforms there would be improved Ctrl+C handling</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 8, 2015 at 1:08 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 subscriber: amccarth.<br>
amccarth added a comment.<br>
<br>
After applying the patch, I get three additional test case failures on Windows.  I'm trying to figure out now which ones.<br>
<br>
Is there something specific I'm supposed to be trying, with regard to Ctrl+C itself.<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>