[Openmp-commits] [PATCH] D95170: [OpenMP] Fix misleading warning for OMP_PLACES

Joachim Protze via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Jan 21 14:55:00 PST 2021


protze.joachim added a comment.

In my quick test, `env KMP_SETTINGS=1 OMP_PROC_BIND=spread ./a.out` reports `OMP_PLACES='cores'`. I could not find the code path that sets places to cores in this case, but I think it would be more consistent to have consistent setting in the unset and the invalid case.

Is there a reason to choose threads?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95170/new/

https://reviews.llvm.org/D95170



More information about the Openmp-commits mailing list