[Openmp-commits] [PATCH] D26860: Fix for D25504 - segfault because of double free()-ing in task deps shutdown code.

Paul Osmialowski via Openmp-commits openmp-commits at lists.llvm.org
Sat Nov 19 14:22:15 PST 2016


pawosm01 accepted this revision.
pawosm01 added a comment.
This revision is now accepted and ready to land.

I tested this change for few hours running superlu_taskdep and strassen_taskdep on different number of cores. Didn't fail, so we can assume it solves the problem.

BTW, don't you think that the task->td_dephash pointer should be NULLified after calling __kmp_dephash_free() on it in kmp_taskdeps.cpp line 380?


Repository:
  rL LLVM

https://reviews.llvm.org/D26860





More information about the Openmp-commits mailing list