[PATCH] D12998: Overhaul OMPT initialization interface

John Mellor-Crummey via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 21 09:57:47 PDT 2015


jmellorcrummey removed rL LLVM as the repository for this revision.
jmellorcrummey updated this revision to Diff 35272.
jmellorcrummey added a comment.

Addressed comments:

- moved the call to ompt_pre_init to after the calls to KMP_DEBUG_ASSERT in __kmp_do_serial_initialize
- moved the call to ompt_post_init before the call to KMP_MB in __kmp_do_serial_initialize
- removed redundant calls to ompt_pre_init/post_init in __kmp_parallel_initialize and __kmp_middle_initialize
- fixed indentation issues caused by tabs
- fixed typos in two comments (a missing "is" in the comment with iff and a misspelled synchronize in a pre-existing comment).


http://reviews.llvm.org/D12998

Files:
  runtime/src/exports_so.txt
  runtime/src/include/30/ompt.h.var
  runtime/src/include/40/ompt.h.var
  runtime/src/include/41/ompt.h.var
  runtime/src/kmp_runtime.c
  runtime/src/ompt-general.c
  runtime/src/ompt-internal.h
  runtime/src/ompt-specific.c
  runtime/src/ompt-specific.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12998.35272.patch
Type: text/x-patch
Size: 15923 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150921/4ee55221/attachment.bin>


More information about the llvm-commits mailing list