[PATCH] D6584: Add per-test timeouts to lit

Jonathan Roelofs jonathan at codesourcery.com
Mon Jul 6 13:00:08 PDT 2015


Sorry for the diff spam.


================
Comment at: utils/lit/lit/util.py:37
@@ +36,3 @@
+            try:
+                os.kill(popen.pid, signal.SIGTERM)
+            except OSError, e:
----------------
I don't know enough about python implementation details to know if this is 'correct' cross-platform-wise, but it appears to work on Darwin, Linux, and Cygwin on Windows.

I got ahold of a Cygwin+Windows machine and was able to test your deadlock reproducer on it with this patch.  I don't have a full development environment on that machine, so I can't run all the other tests that need FileCheck, nor the full llvm testsuite, but this looks promising.


http://reviews.llvm.org/D6584







More information about the llvm-commits mailing list