Page MenuHomePhabricator

Bug fix: removed zeroing of the th.th_task_state for master thread at start of nested parallel.
ClosedPublic

Authored by jlpeyton on Oct 13 2015, 10:54 AM.

Details

Summary

The th.th_task_state for the master thread at the start of a nested parallel should not be zeroed in kmp_allocate_team() because it is later put in the stack of states in kmp_fork_call() for further re-use after exiting the nested region. It is zeroed after being put in the stack.

Diff Detail

Repository
rL LLVM

Event Timeline

jlpeyton updated this revision to Diff 37272.Oct 13 2015, 10:54 AM
jlpeyton retitled this revision from to Bug fix: removed zeroing of the th.th_task_state for master thread at start of nested parallel..
jlpeyton updated this object.
jlpeyton set the repository for this revision to rL LLVM.
jlpeyton added a subscriber: openmp-commits.
AndreyChurbanov accepted this revision.Oct 20 2015, 7:26 AM
AndreyChurbanov edited edge metadata.

LGTM

This revision is now accepted and ready to land.Oct 20 2015, 7:26 AM
This revision was automatically updated to reflect the committed changes.