[Openmp-commits] [PATCH] D49699: [OpenMP][Stats] Cleanup stats gathering code

[Jonathan Peyton via Phabricator via Openmp-commits]

jlpeyton created this revision.
jlpeyton added reviewers: AndreyChurbanov, tlwilmar.
jlpeyton added a project: OpenMP.
Herald added a subscriber: guansong.

1. Remove unnecessary data from list node structure
2. Remove `timerPair` in favor of pushing/popping `explicitTimers`.  This way, nested timers will work properly.
3. Fix `#pragma omp critical` timers
4. Add histogram capability
5. Add `KMP_STATS_FILE` formatting capability
6. Have time partitioned into serial & parallel by introducing `partitionedTimers::exchange()`.  This also counts the number of serial regions in the executable.
7. Fix up the timers around OMP loops so that scheduling overhead and work are both counted correctly.
8. Fix up the iterations statistics so they count the number of iterations the thread receives at each loop scheduling event
9. Change timers so there is only one RDTSC read per event change
10. Fix up the outdated comments for the timers


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D49699

Files:
  runtime/src/kmp_csupport.cpp
  runtime/src/kmp_dispatch.cpp
  runtime/src/kmp_runtime.cpp
  runtime/src/kmp_sched.cpp
  runtime/src/kmp_stats.cpp
  runtime/src/kmp_stats.h
  runtime/src/kmp_stats_timing.cpp
  runtime/src/z_Linux_util.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49699.156884.patch
Type: text/x-patch
Size: 59479 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20180723/9df6d99a/attachment-0001.bin>