[Openmp-commits] [PATCH] D31012: Fix incorrect initial value of __kmp_affinity_type.

Hansang Bae via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Mar 15 16:09:47 PDT 2017


hbae created this revision.

Affinity initialization code expects `__kmp_affinity_type` has the value `affinity_default` by default, but the cleanup code does not properly set the value back to `affinity_default`.
This may introduce some issues when multiple roots are trying to initialize/uninitialize the runtime successively.


Repository:
  rL LLVM

https://reviews.llvm.org/D31012

Files:
  runtime/src/kmp_affinity.cpp


Index: runtime/src/kmp_affinity.cpp
===================================================================
--- runtime/src/kmp_affinity.cpp
+++ runtime/src/kmp_affinity.cpp
@@ -4197,6 +4197,7 @@
         __kmp_affin_fullMask = NULL;
     }
     __kmp_affinity_num_masks = 0;
+    __kmp_affinity_type = affinity_default;
 # if OMP_40_ENABLED
     __kmp_affinity_num_places = 0;
 # endif


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31012.91952.patch
Type: text/x-patch
Size: 383 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20170315/636599f5/attachment.bin>


More information about the Openmp-commits mailing list