[PATCH] D77819: [lit] Add SKIPPED test result category

[Joel E. Denny via Phabricator via llvm-commits]

jdenny added inline comments.


================
Comment at: llvm/utils/lit/tests/max-time.py:3
+#
+# RUN: %{lit} %{inputs}/max-time --max-time=1 2>&1  |  FileCheck %s
+
----------------
yln wrote:
> jdenny wrote:
> > On heavily loaded test systems, is there a chance of a race here?
> Yes, race is between `--max-time=1` and `sleep 5` in [slow.txt].  I will increase 5 to 60 when landing.  That should work for all practical purposes.
Isn't there theoretically a race between `--max-time=1` and fast.txt as well?  I'm not saying you need to change it, but I at least want to be sure I'm not misunderstanding something.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77819/new/

https://reviews.llvm.org/D77819