[all-commits] [llvm/llvm-project] f39a97: [CMake] Include runtimes test suites in check-all

Petr Hosek via All-commits all-commits at lists.llvm.org
Thu Mar 10 10:18:57 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f39a971d821097df1936469b3fd5ba6a9b8e4b69
      https://github.com/llvm/llvm-project/commit/f39a971d821097df1936469b3fd5ba6a9b8e4b69
  Author: Petr Hosek <phosek at google.com>
  Date:   2022-03-10 (Thu, 10 Mar 2022)

  Changed paths:
    M llvm/CMakeLists.txt
    M llvm/runtimes/CMakeLists.txt
    M runtimes/CMakeLists.txt
    A runtimes/Tests.cmake.in

  Log Message:
  -----------
  [CMake] Include runtimes test suites in check-all

Prior to this change, we would make check-all depend on check-runtimes
which is a target that runs tests in the runtimes build. This means that
the runtimes tests are going to run prior to other test suites in
check-all, and if one of them fails, we won't run the other test suites
at all.

To address this issue, we instead collect the list of test suites and
their dependencies from the runtimes subbuild, and include them in
check-all, so a failure of runtimes test suite doesn't prevent other
test suites from being executed.

This addresses https://github.com/llvm/llvm-project/issues/54154.

Differential Revision: https://reviews.llvm.org/D121276




More information about the All-commits mailing list