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

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

What if we add an instances vector argument to`timeTraceProfilerFinishThread`.
Our threads would work as before but when they finish they use the runtime global value "Instances" which was set in the constructor via `getTimeTraceProfilerInstances`.
Net benefit, assuming I got it right, is that we only need one extra argument, one global pointer, and to expose the function.
Afterwards, they should all agree on the instances vector and our problems should be gone, no?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140248