[all-commits] [llvm/llvm-project] 04ce01: Reapply "[llvm][lit] Add option to run only the fa...

[Michael Buch via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 04ce013d7c87c361b14878b0038d41d37290151f
      https://github.com/llvm/llvm-project/commit/04ce013d7c87c361b14878b0038d41d37290151f
  Author: Michael Buch <michaelbuch12 at gmail.com>
  Date:   2025-12-12 (Fri, 12 Dec 2025)

  Changed paths:
    M llvm/docs/CommandGuide/lit.rst
    M llvm/utils/lit/lit/TestTimes.py
    M llvm/utils/lit/lit/cl_arguments.py
    M llvm/utils/lit/lit/main.py
    A llvm/utils/lit/tests/Inputs/filter-failed/fail.txt
    A llvm/utils/lit/tests/Inputs/filter-failed/lit.cfg
    A llvm/utils/lit/tests/Inputs/filter-failed/pass.txt
    A llvm/utils/lit/tests/Inputs/filter-failed/unresolved.txt
    A llvm/utils/lit/tests/Inputs/filter-failed/xfail.txt
    A llvm/utils/lit/tests/Inputs/filter-failed/xpass.txt
    A llvm/utils/lit/tests/filter-failed-delete.py
    A llvm/utils/lit/tests/filter-failed-rerun.py
    A llvm/utils/lit/tests/filter-failed.py

  Log Message:
  -----------
  Reapply "[llvm][lit] Add option to run only the failed tests" (#171588)

This reverts commit 3847648e84d2ff5194f605a8a9a5c0a5e5174939.

Relands https://github.com/llvm/llvm-project/pull/158043 which got
auto-merged on a revision which wasn't approved.

The only addition to the approved version was that we adjust how we set
the time for failed tests. We used to just assign it the negative value
of the elapsed time. But if the test failed with `0` seconds (which some
of the new tests do), we would mark it `-0`. But the check for whether
something failed checks for `time < 0`. That messed with the new
`--filter-failed` option of this PR. This was only an issue on Windows
CI, but presumably can happen on any platform. Happy to do this in a
separate PR.

---- Original PR

This patch adds a new --filter-failed option to llvm-lit, which when
set, will only run the tests that have previously failed.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications