My concern, and the reason I brought it up, is because it seems like unnecessary complexity which only benefits a handful of people.  I don't like adding complexity that isn't useful for a wide audience.  <br><br>If this is just for making your own local test suite terminate in a reasonable amount of time, then you can do that with an external shell script which you run when you notice it's hung.  If the issue is not knowing it's hung, then we should change the test runner to print its output as the test suite runs instead of at the end (we should do this anyway actually because that's very useful).<br><br>And in addition to both of those, we really should be either fixing the tests or disabling them.  Now that this timeout is in, i have a feeling that fixing the hanging tests will be de-prioritized.  <br><br>Anyway, this patch is going in (or has gone in) for now, but I really don't want to see this timeout logic get any more complicated in its current form.<br><div class="gmail_quote">On Thu, Dec 18, 2014 at 5:51 AM Ed Maste <<a href="mailto:emaste@freebsd.org">emaste@freebsd.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In <a href="http://reviews.llvm.org/D6669#101732" target="_blank">http://reviews.llvm.org/D6669#<u></u>101732</a>, @chaoren wrote:<br>
<br>
> Can dosep supply different arguments to different invocations of dotest?<br>
>  Otherwise, every test must run with the same time limit under dosep if we<br>
>  supply timeout as an argument.<br>
<br>
<br>
Although we could solve this with some of the suggestions above, I'm not sure it's that important as long as the global timeout is long enough to accommodate all of the tests and short enough that the wait is reasonable if there's a stuck test. 5 minutes seems like it would accomplish that.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D6669" target="_blank">http://reviews.llvm.org/D6669</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote></div>