Task Team Synchronization Fix

[jlpeyton]

Summary

The fix syncs up the new threads to have the same task_state and task_team as the old threads. The master thread is skipped, because it shouldn't at this point have the team's task_team value yet -- it should still have parent_team's task_team. It gets pointed at the new team's task_team later, after __kmp_allocate_team returns, and the master has stored a memo of it's old task_state.

[tlwilmar]

See comment in code.

runtime/src/kmp_runtime.c

5065

This has a bug that was fixed later. Just take the two lines from the loop below, and add them to the loop above, after the call to __kmp_initialize_info. Then the comment and (now empty) loop below can be removed.

[jlpeyton]

Addressed Terry's comment.

[tlwilmar]

LGTM

[jlpeyton]

Committed revision 237916.

I forgot to mention the Differential Revision in commit message.

runtime/src/kmp_runtime.c

#endif // KMP_NESTED_HOT_TEAMS
                                            __kmp_gtid_from_tid( f, team ) );
                 }
             }
             else {
                 int old_state = team->t.t_threads[0]->th.th_task_state;
                 for (f=0;  f < team->t.t_nproc; ++f) {
                     __kmp_initialize_info( team->t.t_threads[ f ], team, f, __kmp_gtid_from_tid( f, team ) );
                     team->t.t_threads[f]->th.th_task_state = old_state;
+                    team->t.t_threads[f]->th.th_task_team = team->t.t_task_team[old_state];
                 }
             }
 
 #ifdef KMP_DEBUG
             for ( f = 0; f < team->t.t_nproc; ++ f ) {
                 KMP_DEBUG_ASSERT( team->t.t_threads[f] &&
                     team->t.t_threads[f]->th.th_team_nproc == team->t.t_nproc );
             }
 #endif