[llvm-commits] [patch] Used TimedExec.sh everywhere

[Dan Gohman]

On Jan 14, 2010, at 2:57 PM, Rafael Espindola wrote:

>> I gave this a shot:
>> 
>>   TimedExec.sh 1 . sleep 10
>> 
>> and found that it slept for 10 seconds, not 1. I think we may need to change
>> how TimedExec.sh works to make it support Linux.
> 
> Yes, I noticed that too. The attached patch fixes it by using the CONT
> signal instead of INFO. I tested that it does kill after the specified
> timeout.

Why SIGCONT? Wouldn't SIGTERM be more conventional? Or, would
SIGKILL be best, since you may not necessarily trust the
application's signal handlers to quit promptly?

Dan