[Openmp-commits] [PATCH] D35860: Introduce KMP_DEVICE_THREAD_LIMIT

Terry Wilmarth via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Jul 25 15:18:50 PDT 2017


tlwilmar created this revision.

This change drops in KMP_DEVICE_THREAD_LIMIT to replace KMP_MAX_THREADS.  It's possible there will eventually be a OMP_DEVICE_THREAD_LIMIT, and we need something to distinguish from OMP_THREAD_LIMIT, which is currently implemented wrong (the fix for that will be added soon in a separate patch). KMP_ALL_THREADS is deprecated here, but we can keep the "all" option on KMP_DEVICE_THREAD_LIMIT to support that functionality. KMP_DEVICE_THREAD_LIMIT now has priority over its deprecated rival KMP_ALL_THREADS. I also cleaned up some comments that incorrectly referred to non-existent __kmp_max_threads variable instead of __kmp_max_nth.

      

I've left the name of where this setting eventually ends up as __kmp_max_nth, for now.

      

This change does not change much in the way of functionality. It does NOT change OMP_THREAD_LIMIT.  It's just cleaning up and setting up for that.


Repository:
  rL LLVM

https://reviews.llvm.org/D35860

Files:
  runtime/src/kmp_runtime.cpp
  runtime/src/kmp_settings.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35860.108165.patch
Type: text/x-patch
Size: 6142 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20170725/3af7a836/attachment.bin>


More information about the Openmp-commits mailing list