[PATCH] D14706: [lit] Implement support of per test timeout in lit.

Dan Liew via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 18 01:39:25 PST 2015


Hi,

On 17 November 2015 at 20:10, Chris Matthews <chris.matthews at apple.com> wrote:
> I don’t think the XUnit files need to be a special case.  Just make sure whatever is in the stack trace section is useful for diagnosing and fixing the problem; specifically, a timeout of X was triggered while running Y.

By "stack trace" I presume you mean the list of executed commands and
their output (i.e. exit code, stdout and stderr). In that respect my
patch does a poor job because if a timeout is triggered the user
**only** gets told that a particular test reached the timeout. It is
definitely desirable to show more information than that when a timeout
occurs so I will need to spend some time redoing that aspect of the
patch.

Dan.


More information about the llvm-commits mailing list