[Openmp-commits] [openmp] r271581 - Merging r270884:

[Jonathan Peyton via Openmp-commits]

Author: jlpeyton
Date: Thu Jun  2 15:32:40 2016
New Revision: 271581

URL: http://llvm.org/viewvc/llvm-project?rev=271581&view=rev
Log:
Merging r270884:
------------------------------------------------------------------------
r270884 | jlpeyton | 2016-05-26 13:19:10 -0500 (Thu, 26 May 2016) | 11 lines

Make LIBOMP_USE_ITT_NOTIFY a setting that can be enabled or disabled

On Blue Gene/Q, having LIBOMP_USE_ITT_NOTIFY support compiled into a
statically-linked binary causes a failure at runtime because dlopen fails.
This patch changes LIBOMP_USE_ITT_NOTIFY to a cacheable configuration setting
that can be disabled.

Patch by John Mellor-Crummey

Differential Revision: http://reviews.llvm.org/D20517

------------------------------------------------------------------------

Modified:
    openmp/branches/release_38/   (props changed)
    openmp/branches/release_38/runtime/CMakeLists.txt
    openmp/branches/release_38/runtime/src/kmp_config.h.cmake
    openmp/branches/release_38/runtime/src/kmp_csupport.c
    openmp/branches/release_38/runtime/src/kmp_runtime.c

Propchange: openmp/branches/release_38/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Thu Jun  2 15:32:40 2016
@@ -1 +1 @@
-/openmp/trunk:257833,258169,258528,258695,258990,263651,264166,270464,270821
+/openmp/trunk:257833,258169,258528,258695,258990,263651,264166,270464,270821,270884

Modified: openmp/branches/release_38/runtime/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/openmp/branches/release_38/runtime/CMakeLists.txt?rev=271581&r1=271580&r2=271581&view=diff
==============================================================================
--- openmp/branches/release_38/runtime/CMakeLists.txt (original)
+++ openmp/branches/release_38/runtime/CMakeLists.txt Thu Jun  2 15:32:40 2016
@@ -192,8 +192,9 @@ elseif("${libomp_build_type_lowercase}"
   set(MINSIZEREL_BUILD TRUE)
 endif()
 
-# Include itt notify interface? Right now, always.
-set(LIBOMP_USE_ITT_NOTIFY TRUE)
+# Include itt notify interface?
+set(LIBOMP_USE_ITT_NOTIFY TRUE CACHE BOOL
+  "Enable ITT notify?")
 
 # normal, profile, stubs library.
 set(NORMAL_LIBRARY FALSE)
@@ -326,6 +327,7 @@ if(${LIBOMP_STANDALONE_BUILD})
   libomp_say("Build                -- ${LIBOMP_BUILD}")
   libomp_say("Use Stats-gathering  -- ${LIBOMP_STATS}")
   libomp_say("Use Debugger-support -- ${LIBOMP_USE_DEBUGGER}")
+  libomp_say("Use ITT notify       -- ${LIBOMP_USE_ITT_NOTIFY}")
   libomp_say("Use OMPT-support     -- ${LIBOMP_OMPT_SUPPORT}")
   if(${LIBOMP_OMPT_SUPPORT})
     libomp_say("Use OMPT-blame       -- ${LIBOMP_OMPT_BLAME}")

Modified: openmp/branches/release_38/runtime/src/kmp_config.h.cmake
URL: http://llvm.org/viewvc/llvm-project/openmp/branches/release_38/runtime/src/kmp_config.h.cmake?rev=271581&r1=271580&r2=271581&view=diff
==============================================================================
--- openmp/branches/release_38/runtime/src/kmp_config.h.cmake (original)
+++ openmp/branches/release_38/runtime/src/kmp_config.h.cmake Thu Jun  2 15:32:40 2016
@@ -85,7 +85,7 @@
 #define KMP_ADJUST_BLOCKTIME 1
 #define BUILD_PARALLEL_ORDERED 1
 #define KMP_ASM_INTRINS 1
-#define USE_ITT_BUILD 1
+#define USE_ITT_BUILD LIBOMP_USE_ITT_NOTIFY
 #define INTEL_ITTNOTIFY_PREFIX __kmp_itt_
 #if ! KMP_MIC
 # define USE_LOAD_BALANCE 1

Modified: openmp/branches/release_38/runtime/src/kmp_csupport.c
URL: http://llvm.org/viewvc/llvm-project/openmp/branches/release_38/runtime/src/kmp_csupport.c?rev=271581&r1=271580&r2=271581&view=diff
==============================================================================
--- openmp/branches/release_38/runtime/src/kmp_csupport.c (original)
+++ openmp/branches/release_38/runtime/src/kmp_csupport.c Thu Jun  2 15:32:40 2016
@@ -511,7 +511,7 @@ __kmpc_end_serialized_parallel(ident_t *
 
    // we need to wait for the proxy tasks before finishing the thread
    if ( task_team != NULL && task_team->tt.tt_found_proxy_tasks )
-        __kmp_task_team_wait(this_thr, serial_team, NULL ); // is an ITT object needed here?
+        __kmp_task_team_wait(this_thr, serial_team USE_ITT_BUILD_ARG(NULL) ); // is an ITT object needed here?
    #endif
 
     KMP_MB();

Modified: openmp/branches/release_38/runtime/src/kmp_runtime.c
URL: http://llvm.org/viewvc/llvm-project/openmp/branches/release_38/runtime/src/kmp_runtime.c?rev=271581&r1=271580&r2=271581&view=diff
==============================================================================
--- openmp/branches/release_38/runtime/src/kmp_runtime.c (original)
+++ openmp/branches/release_38/runtime/src/kmp_runtime.c Thu Jun  2 15:32:40 2016
@@ -3804,7 +3804,9 @@ __kmp_register_root( int initial_thread
     /* prepare the master thread for get_gtid() */
     __kmp_gtid_set_specific( gtid );
 
+#if USE_ITT_BUILD
     __kmp_itt_thread_name( gtid );
+#endif /* USE_ITT_BUILD */
 
     #ifdef KMP_TDATA_GTID
         __kmp_gtid = gtid;
@@ -3972,7 +3974,7 @@ __kmp_unregister_root_current_thread( in
         // the runtime is shutting down so we won't report any events
         thread->th.ompt_thread_info.state = ompt_state_undefined;
 #endif
-        __kmp_task_team_wait(thread, team, NULL );
+        __kmp_task_team_wait(thread, team USE_ITT_BUILD_ARG(NULL));
    }
 #endif