<div dir="ltr">I would love to have this feature, so +1 on that.<div><br></div><div>I would prefer that it not introduce a new failure code though, and just use FAIL with some extra message output.</div><div><br></div><div>It might be worth considering whether or not we should run the subprocesses in their own process group if this is enabled, so that we can kill the entire process group. I'm not sure if there is any downside to that, and I would like to guarantee that if we have timeouts we never accidentally leave a stray runaway subprocess, which might otherwise bog down a CI setup.</div><div><br></div><div>It looks like you are doing some more work on this so I will hold further review until then.</div><div><br></div><div> - Daniel</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 9, 2014 at 1:58 PM, Jonathan Roelofs <span dir="ltr"><<a href="mailto:jonathan@codesourcery.com" target="_blank">jonathan@codesourcery.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi ddunbar, EricWF,<br>
<br>
This is different than the option '--max-time', which limits total testing time.<br>
<br>
<a href="http://reviews.llvm.org/D6584" target="_blank">http://reviews.llvm.org/D6584</a><br>
<br>
Files:<br>
  lit/LitConfig.py<br>
  lit/Test.py<br>
  lit/TestRunner.py<br>
  lit/formats/base.py<br>
  lit/main.py<br>
  lit/run.py<br>
  lit/util.py<br>
</blockquote></div><br></div>