[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