OpenMPToolsInterface / LLVM-openmp

This is a stale repository and only there for the commit history. All development moved over to the llvm-project repository! Was: LLVM OpenMP runtime with experimental changes for OMPT (Preview of TR7 features in *_tr branches)
Other
16 stars 13 forks source link

[OMPD] The LLVM OMPD library has the wrong names for the "release" functions #55

Open jdelsign opened 5 years ago

jdelsign commented 5 years ago

The OpenMP v5.0 spec (and OpenMP omp-tools.h header) use "ompdrel" as the prefix for the "release" functions. However, the LLVM OMPD library uses "ompdrelease" as the prefix for those functions (which is what OMPD 3.1 used). The OMPD DLL defines the following functions:

  ray23{jdelsign}103: nm /collab/usr/global/tools/ompd/ppc64le-ray//lib/libompd.so | grep ompd_rel
  000000000001c100 T ompd_release_address_space_handle
  00000000000206f0 T ompd_release_display_control_vars
  000000000001df20 T ompd_release_parallel_handle
  000000000001ef30 T ompd_release_task_handle
  000000000001c8d0 T ompd_release_thread_handle
  ray23{jdelsign}104: 

All of those functions should be renamed to conform to the OMP v5.0 standard.