[Openmp-commits] [openmp] [libomp] Fix hwloc include for non-standard paths (PR #184087)

[Robert Imschweiler via Openmp-commits]

================
@@ -189,6 +189,14 @@ else()
   set(LIBOMP_LINKER_LANGUAGE CXX)
 endif()
 if(${LIBOMP_USE_HWLOC})
+  target_include_directories(obj.omp
+                             PUBLIC
+			     "$<BUILD_INTERFACE:${LIBOMP_HWLOC_INCLUDE_DIR}>"
+			     "$<INSTALL_INTERFACE:${LIBOMP_HWLOC_INCLUDE_DIR}>"
+  )
+  # omp itself uses obj.omp and thus doesn't need to include hwloc headers
+  # itself. However, we want to propagate the include to consumers of omp (e.g.
+  # libompd).
----------------
ro-i wrote:

I think there was a misunderstanding: I meant that omp as a target doesn't need the include because there is no compilation involved for omp anymore (since omp is built using obj.omp). What wording would you suggest to clarify that?

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