[Openmp-commits] [openmp] Reland: [OpenMP] Add ompTest library to OpenMP (PR #154786)

[via Openmp-commits]

================
@@ -15,6 +15,21 @@ if(LIBOMP_OMPT_SUPPORT AND LIBOMP_ARCHER_SUPPORT)
   target_link_libraries(archer ${CMAKE_DL_LIBS})
   add_library(archer_static STATIC ompt-tsan.cpp)
 
+  if(TARGET cxx-headers)
+    add_dependencies(archer cxx-headers)
+    add_dependencies(archer_static cxx-headers)
+  endif()
+
+  if(TARGET cxx_shared)
+    add_dependencies(archer cxx_shared)
+    add_dependencies(archer_static cxx_shared)
+  endif()
+
+  if(TARGET cxxabi_shared)
+    add_dependencies(archer cxxabi_shared)
+    add_dependencies(archer_static cxxabi_shared)
+  endif()
----------------
jprotze wrote:

The lines are there to establish a build ordering in the case that libcxx and openmp are both built as RUNTIME (and clang is configured to use libc++). If generate-cxx-headers is processed in parallel to building archer, the compiler might see a half-written C++ header file and error out.

It's not about forcing to link against the in-tree libc++, but it might be the only libc++ available.

https://github.com/llvm/llvm-project/pull/154786