[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