[Openmp-commits] [openmp] [OpenMP] Fix preprocessor mismatches between include and usages of hwloc (PR #158349)

Michael Klemm via Openmp-commits openmp-commits at lists.llvm.org
Fri Sep 26 12:45:02 PDT 2025


https://github.com/mjklemm requested changes to this pull request.

I think `KMP_INCLUDES_HWLOC` does not correctly describe what it actually means. `KMP_USE_HWLOC` is to enable hwloc support in general, bearing in mind that additional conditions may disable it again.  I think `KMP_HWLOC_ENABLED` might be a better choice, because it then describes that a) `hwloc.h` has been included, and b) that the hwloc-based affinity code is also enabled.

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


More information about the Openmp-commits mailing list