[Openmp-commits] [PATCH] D55795: Fix for CMPLRS-48319 proc_bind affinity bug

[Jonathan Peyton via Phabricator via Openmp-commits]

jlpeyton created this revision.
jlpeyton added reviewers: AndreyChurbanov, tlwilmar, hbae, omalyshe.
jlpeyton added a project: OpenMP.
Herald added a subscriber: jfb.

Using proc_bind clause on a nested #pragma omp parallel region
with KMP_AFFINITY set causes an assertion error. This assertion occurs because
the place-partition-var is not properly initialized in the nested master threads.
Trying to get an intuitive result with KMP_AFFINITY + proc_bind is difficult
because of how the KMP_AFFINITY gtid-to-place mapping occurs. This
patch creates an initial place list no matter what affinity mechanism is used.
For KMP_AFFINITY, the place-partition-var is initialized to all the places.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D55795

Files:
  runtime/src/kmp_affinity.cpp
  runtime/src/kmp_ftn_entry.h
  runtime/test/affinity/bug-nested.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55795.178541.patch
Type: text/x-patch
Size: 3030 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20181217/101e0b7e/attachment.bin>