[Openmp-commits] [openmp] 5f97e19 - [OpenMP][OMPT] Remove Threads dependency from omptest (#185930)
via Openmp-commits
openmp-commits at lists.llvm.org
Thu Mar 12 05:08:53 PDT 2026
Author: Michael Halkenhäuser
Date: 2026-03-12T13:08:49+01:00
New Revision: 5f97e191b34f4fd6185a9caa7468aa0432c5d503
URL: https://github.com/llvm/llvm-project/commit/5f97e191b34f4fd6185a9caa7468aa0432c5d503
DIFF: https://github.com/llvm/llvm-project/commit/5f97e191b34f4fd6185a9caa7468aa0432c5d503.diff
LOG: [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.
Added:
Modified:
openmp/tools/omptest/CMakeLists.txt
Removed:
################################################################################
diff --git a/openmp/tools/omptest/CMakeLists.txt b/openmp/tools/omptest/CMakeLists.txt
index 9ff71f49b0abd..daf3d769569fa 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
More information about the Openmp-commits
mailing list