[Openmp-commits] [PATCH] D29638: Fix for bug https://llvm.org/bugs/show_bug.cgi?id=30889

[Andrey Churbanov via Phabricator via Openmp-commits]

AndreyChurbanov added inline comments.


================
Comment at: runtime/src/kmp_runtime.cpp:2050-2051
 #if 0
         // 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]);
----------------
grokos wrote:
> What about here?
This is a different problem related with inconsistent references to task team in thread and team structures. It was actually fixed by the commit r249711 (https://reviews.llvm.org/D13353).

I will submit separate patch to re-enable this assertion.



Repository:
  rL LLVM

https://reviews.llvm.org/D29638