From 5f97e191b34f4fd6185a9caa7468aa0432c5d503 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Michael=20Halkenh=C3=A4user?= Date: Thu, 12 Mar 2026 13:08:49 +0100 Subject: [PATCH] [OpenMP][OMPT] Remove Threads dependency from omptest (#185930) Removed link against `Threads`. Reason: it is potentially problematic and optional. The issue would manifest, if `omptest` is used via `find_package`. But `Threads` might not be found and cause a link error. --- openmp/tools/omptest/CMakeLists.txt | 4 ---- 1 file changed, 4 deletions(-) diff --git a/openmp/tools/omptest/CMakeLists.txt b/openmp/tools/omptest/CMakeLists.txt index 9ff71f49b0ab..daf3d769569f 100644 --- a/openmp/tools/omptest/CMakeLists.txt +++ b/openmp/tools/omptest/CMakeLists.txt @@ -87,10 +87,6 @@ if ((NOT LIBOMPTEST_BUILD_STANDALONE) OR LIBOMPTEST_BUILD_UNITTESTS) # available to dependant targets, e.g. for unit tests. target_link_libraries(omptest PUBLIC default_gtest) - # Link against Threads (recommended for GTest). - find_package(Threads REQUIRED) - target_link_libraries(omptest PUBLIC Threads::Threads) - # Ensure that embedded GTest symbols are exported from libomptest.so even in # builds that default to hidden. set_target_properties(omptest PROPERTIES