[PATCH] D11062: Bug fixes in OMPT support

[John Mellor-Crummey]

jmellorcrummey created this revision.
jmellorcrummey added a reviewer: jcownie.
jmellorcrummey added a subscriber: llvm-commits.
jmellorcrummey set the repository for this revision to rL LLVM.

1. in kmp_csupport.c,
       move computation of parameters only needed for OMPT tracing inside a conditional to reduce overhead 
       if not receiving ompt_event_master_begin callbacks.
2. in kmp_gsupport.c
      - remove spurious reset of OMPT reenter_runtime_frame (which is set in its caller, GOMP_parallel_start
      - correct placement of #if OMP_TRACE so that state is maintained even if tracing support not included.
3. in z_Linux_util.c
      - add architecture independent support for OMPT by setting and resetting
        OMPT's exit_frame_ptr before and after invoking a microtask.
4.  On the Intel MIC, the loader refuses to retain static symbols in the libomp.so shared library, even though 
     tools need them. The loader could not be bullied into doing so. To accommodate this, I changed the 
     visibility of OMPT placeholder functions to public. This required additions in exports.so.txt, adding 
     extern "C" scoping in ompt-general.c so that the public placeholder symbols won't be mangled.



Repository:
  rL LLVM

http://reviews.llvm.org/D11062

Files:
  runtime/src/exports_so.txt
  runtime/src/kmp_csupport.c
  runtime/src/kmp_gsupport.c
  runtime/src/ompt-general.c
  runtime/src/z_Linux_util.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11062.29331.patch
Type: text/x-patch
Size: 6667 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150709/5537cea3/attachment.bin>