[Openmp-commits] [openmp] [OpenMP][OMPT] Remove Threads dependency from omptest (PR #185930)
Michael Halkenhäuser via Openmp-commits
openmp-commits at lists.llvm.org
Wed Mar 11 10:16:13 PDT 2026
https://github.com/mhalk created https://github.com/llvm/llvm-project/pull/185930
Removed link against `Threads`.
Reason: it is potentially problematic and optional.
The issue would manifest, if `omptest` is used via `find_package`.
>From 53682102e98f86cf590b1f62492f225e2da97ab2 Mon Sep 17 00:00:00 2001
From: Michael Halkenhaeuser <MichaelGerald.Halkenhauser at amd.com>
Date: Wed, 11 Mar 2026 12:07:50 -0500
Subject: [PATCH] [OpenMP][OMPT] Remove Threads dependency from omptest
Removed link against Threads.
Reason: it is potentially problematic and optional.
The issue would manifest, if omptest is used via find_package.
---
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 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