From 77e019c23389b508d75d57653069868a2812e5c7 Mon Sep 17 00:00:00 2001 From: Michael Kruse Date: Sun, 12 Dec 2021 10:29:35 -0600 Subject: [PATCH] [OpenMP] Add "not" to test dependencies. The `not` program is used to test executions prefixed with `%libomptarget-run-fail-`. Currently `not` is not used for libomp tests, but might be used in the future and its dependency does not add any additional burden over the already established `FileCheck` dependency. Required to add libomptarget testing to the Phabricator pre-merge check (see https://github.com/google/llvm-premerge-checks/issues/368) Reviewed By: jdenny, JonChesterfield Differential Revision: https://reviews.llvm.org/D115454 --- openmp/cmake/OpenMPTesting.cmake | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/openmp/cmake/OpenMPTesting.cmake b/openmp/cmake/OpenMPTesting.cmake index 6a6d1cb..81732ce 100644 --- a/openmp/cmake/OpenMPTesting.cmake +++ b/openmp/cmake/OpenMPTesting.cmake @@ -196,14 +196,14 @@ function(add_openmp_testsuite target comment) ${comment} ${ARG_UNPARSED_ARGUMENTS} EXCLUDE_FROM_CHECK_ALL - DEPENDS clang FileCheck ${ARG_DEPENDS} + DEPENDS clang FileCheck not ${ARG_DEPENDS} ARGS ${ARG_ARGS} ) else() add_lit_testsuite(${target} ${comment} ${ARG_UNPARSED_ARGUMENTS} - DEPENDS clang FileCheck ${ARG_DEPENDS} + DEPENDS clang FileCheck not ${ARG_DEPENDS} ARGS ${ARG_ARGS} ) endif() -- 2.7.4