<p dir="ltr">>RE: Is change this necessary to ensure correctness, or is it just a matter of saving some lines of code?<br>
It was done to ensure correctness because Target::Launch consists of ~4 branches of code where it was needed and it was to difficult to maintain. In addition it is needed only for lldb to ensure that event is handled before printing of (lldb) prompt, but if it doesn't matter when we using a public API (i.e. SBTarget::Launch).<br>
And the last benefit is to remove the duplicated code.</p>
<p dir="ltr">>RE: It's not clear to me why is this the case, but the test takes ~6 minutes now, where previously it took slightly less than 5.<br>
I think it is a matter of TestMiStartupOptoins failure because each test is waiting ~30sec before failing.</p>
<p dir="ltr">Thanks, <br>
Ilia </p>
<div class="gmail_quote">On May 7, 2015 4:40 PM, "Pavel Labath" <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Is change this necessary to ensure correctness, or is it just a matter of saving some lines of code?<br>
<br>
I am asking this because, this change has increased the running time of the test suite (compare <a href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/2252" target="_blank">http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/2252</a> with <a href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/2251" target="_blank">http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/2251</a>), sending TestConcurrentEvents over the 5m limit on the linux build bot. It's not clear to me why is this the case, but the test takes ~6 minutes now, where previously it took slightly less than 5. For now I have increased the timeout limit to address this, but we will need to do something different in the future.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D9373" target="_blank">http://reviews.llvm.org/D9373</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div>