[Openmp-commits] [PATCH] D100181: [OpenMP] [OMPD] [1/6] Implementation of OMPD debugging library - libompd. Code changes in openmp/runtime to support libompd.

Vignesh Balasubramanian via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri Jun 4 04:58:54 PDT 2021


Vigneshbalu marked an inline comment as done.
Vigneshbalu added inline comments.


================
Comment at: openmp/runtime/src/kmp_runtime.cpp:1344
+#if OMPD_SUPPORT
+    OMPT_CUR_TASK_INFO(this_thr)->scheduling_parent =
+        this_thr->th.th_current_task->td_parent;
----------------
hbae wrote:
> Vigneshbalu wrote:
> > hbae wrote:
> > > Is there any reason to keep this for serialized parallel?
> > Apologies for the oversight, removed the wrong assignments.  Working on the changes, will update the diff.
> I think other changes are OK. This one is also need to be removed.
Removed  and updated the diff, please let us know if there is any other review comments.


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

https://reviews.llvm.org/D100181



More information about the Openmp-commits mailing list