[Openmp-commits] [PATCH] D70412: [OpenMP][Tool] Runtime warning for missing TSan-option

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri Nov 29 10:59:20 PST 2019


jdoerfert added a comment.

In D70412#1763950 <https://reviews.llvm.org/D70412#1763950>, @protze.joachim wrote:

> The conclusion of the discussion in the intel forum thread was, that the call from `__kmp_resume` to `__kmp_suspend_initialize_thread` is superflous, because the thread will call `__kmp_suspend_initialize_thread` always during the own initialization before changing a variable to signal readiness.


But that doesn't make this obsolete?

This looks generally fine to me I just want to understand why we can't fix the runtime instead.


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

https://reviews.llvm.org/D70412





More information about the Openmp-commits mailing list