[Openmp-commits] [openmp] r314242 - Remove unused t_single_lock

Jonathan Peyton via Openmp-commits openmp-commits at lists.llvm.org
Tue Sep 26 13:12:17 PDT 2017


Author: jlpeyton
Date: Tue Sep 26 13:12:16 2017
New Revision: 314242

URL: http://llvm.org/viewvc/llvm-project?rev=314242&view=rev
Log:
Remove unused t_single_lock

Add padding inside team structure to keep same structure size.

Modified:
    openmp/trunk/runtime/src/kmp.h
    openmp/trunk/runtime/src/kmp_runtime.cpp

Modified: openmp/trunk/runtime/src/kmp.h
URL: http://llvm.org/viewvc/llvm-project/openmp/trunk/runtime/src/kmp.h?rev=314242&r1=314241&r2=314242&view=diff
==============================================================================
--- openmp/trunk/runtime/src/kmp.h (original)
+++ openmp/trunk/runtime/src/kmp.h Tue Sep 26 13:12:16 2017
@@ -2536,7 +2536,7 @@ typedef struct KMP_ALIGN_CACHE kmp_base_
   KMP_ALIGN_CACHE kmp_ordered_team_t t_ordered;
   kmp_balign_team_t t_bar[bs_last_barrier];
   volatile int t_construct; // count of single directive encountered by team
-  kmp_lock_t t_single_lock; // team specific lock
+  char pad[sizeof(kmp_lock_t)]; // padding to maintain performance on big iron
 
   // Master only
   // ---------------------------------------------------------------------------

Modified: openmp/trunk/runtime/src/kmp_runtime.cpp
URL: http://llvm.org/viewvc/llvm-project/openmp/trunk/runtime/src/kmp_runtime.cpp?rev=314242&r1=314241&r2=314242&view=diff
==============================================================================
--- openmp/trunk/runtime/src/kmp_runtime.cpp (original)
+++ openmp/trunk/runtime/src/kmp_runtime.cpp Tue Sep 26 13:12:16 2017
@@ -4444,7 +4444,6 @@ static void __kmp_initialize_team(kmp_te
 #endif /* KMP_ARCH_X86 || KMP_ARCH_X86_64 */
 
   team->t.t_construct = 0;
-  __kmp_init_lock(&team->t.t_single_lock);
 
   team->t.t_ordered.dt.t_value = 0;
   team->t.t_master_active = FALSE;




More information about the Openmp-commits mailing list