[Openmp-commits] [PATCH] D16714: [OMPT] Fix wrong parent_task_id in serialized parallel_begin with GCC
Jonas Hahnfeld via Openmp-commits
openmp-commits at lists.llvm.org
Mon Feb 29 00:21:31 PST 2016
Hahnfeld added a comment.
In http://reviews.llvm.org/D16714#362790, @omalyshe wrote:
> Is it OK that __ompt_parallel_id_new() is called several times for one region? First in line 393 and then inside __kmp_serialized_parallel() it can be called once or twice.
While this isn't good from a performance perspective I think this won't hurt for correctness: It will skip some ids but contiguous ids aren't required by the standard.
On the other hand I don't know if this may be required in some cases, I don't always fully understand the code paths in the runtime library...
http://reviews.llvm.org/D16714
More information about the Openmp-commits
mailing list