[PATCH] D140248: [OpenMP] Enable profiling on multiple threads

Mark Dewing via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 16 14:20:33 PST 2022


markdewing created this revision.
Herald added subscribers: guansong, hiraditya, yaxunl.
Herald added a project: All.
markdewing requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: llvm-commits, openmp-commits, sstefan1.
Herald added projects: OpenMP, LLVM.

This patch fixes the issue where the built-in profiler doesn't produce data for multiple threads:
https://github.com/llvm/llvm-project/issues/57985

Before creating a TimeScope object, the code checks to see if a profiler instance has
been created for that thread.  If not, it creates one and saves a pointer for later.
At shutdown, all the saved profiler instances are finished before the call to write out the data.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D140248

Files:
  llvm/include/llvm/Support/TimeProfiler.h
  llvm/lib/Support/TimeProfiler.cpp
  openmp/libomptarget/src/private.h
  openmp/libomptarget/src/rtl.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D140248.483671.patch
Type: text/x-patch
Size: 4440 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221216/a75786fe/attachment.bin>


More information about the llvm-commits mailing list