[Openmp-commits] [PATCH] D29638: Fix for bug https://llvm.org/bugs/show_bug.cgi?id=30889
George Rokos via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Tue Feb 7 14:16:59 PST 2017
grokos added a comment.
I am not an expert here and I'm not sure exactly how this works, but it seems to fix the problem.
One question though: The same assertion can be found throughout `kmp_runtime.cpp`, do other instances need this patching as well? E.g. line 2050, where there is a comment about a forthcoming patch that fixes the preconditions which spuriously make assertions to trigger.
Comment at: runtime/src/kmp_runtime.cpp:2050-2051
// Patch out an assertion that trips while the runtime seems to operate correctly.
// Avoiding the preconditions that cause the assertion to trip has been promised as a forthcoming patch.
KMP_DEBUG_ASSERT(master_th->th.th_task_team == parent_team->t.t_task_team[master_th->th.th_task_state]);
What about here?
More information about the Openmp-commits