[Openmp-commits] [PATCH] D11759: Tidy statistics collection

Jonathan Peyton via Openmp-commits openmp-commits at lists.llvm.org
Tue Aug 11 14:38:50 PDT 2015


jlpeyton added a comment.

LGTM too.


================
Comment at: runtime/src/z_Linux_util.c:1904
@@ -1903,3 +1903,3 @@
 {
-    KMP_TIME_BLOCK(USER_resume);
+    KMP_TIME_DEVELOPER_BLOCK(USER_resume);
     int status;
----------------
AndreyChurbanov wrote:
> Should this counter be placed in kmp_resume_template instead?  The kmp_resume_monitor works at shutdown time, that does not correlate well with "USER_resume" event.
I've moved this to kmp_resume_template() in the committed version.


Repository:
  rL LLVM

http://reviews.llvm.org/D11759





More information about the Openmp-commits mailing list