[Openmp-commits] [PATCH] D64571: [OPENMP]Fix threadid in __kmpc_omp_taskwait call for dependent target calls.

[Alexey Bataev via Phabricator via Openmp-commits]

ABataev added a comment.

In D64571#2162397 <https://reviews.llvm.org/D64571#2162397>, @saiislam wrote:

> In D64571#2162388 <https://reviews.llvm.org/D64571#2162388>, @ABataev wrote:
>
> > In D64571#2160362 <https://reviews.llvm.org/D64571#2160362>, @protze.joachim wrote:
> >
> > > The test fails for me with segfault in line 22.
> > >
> > >   $ gdb target_depend_nowait.cpp.tmp-x86_64-pc-linux-gnu
> > >   (gdb) run
> > >   ...
> > >   Program received signal SIGSEGV, Segmentation fault.
> > >   (gdb) bt
> > >   #0  0x0000000000400fcf in .omp_outlined._debug__ (.global_tid.=0x2aaab96b9be0, .bound_tid.=0x2aaab96b9bd8) at target_depend_nowait.cpp:22
> > >   #1  0x0000000000401e8d in .omp_outlined..23 (.global_tid.=0x2aaab96b9be0, .bound_tid.=0x2aaab96b9bd8) at target_depend_nowait.cpp:18
> > >   #2  0x00002aaaab574213 in __kmp_invoke_microtask () from libomp.so
> > >   #3  0x00002aaaab51338e in __kmp_invoke_task_func () from libomp.so
> > >   #4  0x00002aaaab5126bf in __kmp_launch_thread () from libomp.so
> > >   #5  0x00002aaaab55d3d0 in __kmp_launch_worker(void*) () from libomp.so
> > >   #6  0x00002aaaabbd6e65 in start_thread () from libpthread.so.0
> > >   #7  0x00002aaaabee988d in clone () from libc.so.6
> > >
> >
> >
> > It is not the problem of this patch, looks like support for mappers causes failures
>
>
> Yes, I can confirm that this test is failing after D68100 <https://reviews.llvm.org/D68100> (declare mapper runtime implementation) for me as well.


Georgios, could you check this?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64571/new/

https://reviews.llvm.org/D64571