[Openmp-commits] [openmp] r296525 - Removing couple unnecessary architecture guards.

Jonathan Peyton via Openmp-commits openmp-commits at lists.llvm.org
Tue Feb 28 13:43:29 PST 2017


Author: jlpeyton
Date: Tue Feb 28 15:43:28 2017
New Revision: 296525

URL: http://llvm.org/viewvc/llvm-project?rev=296525&view=rev
Log:
Removing couple unnecessary architecture guards.

This section of code (__kmp_test_then_* functions) is guarded by
(KMP_ARCH_X86 || KMP_ARCH_X86_64) so it does not make sense to have other
architecture guards inside this section.  Non-x86 architectures always
use intrinsics (__sync_*)

Modified:
    openmp/trunk/runtime/src/z_Linux_util.cpp

Modified: openmp/trunk/runtime/src/z_Linux_util.cpp
URL: http://llvm.org/viewvc/llvm-project/openmp/trunk/runtime/src/z_Linux_util.cpp?rev=296525&r1=296524&r2=296525&view=diff
==============================================================================
--- openmp/trunk/runtime/src/z_Linux_util.cpp (original)
+++ openmp/trunk/runtime/src/z_Linux_util.cpp Tue Feb 28 15:43:28 2017
@@ -428,7 +428,7 @@ __kmp_test_then_and32( volatile kmp_int3
     return old_value;
 }
 
-# if KMP_ARCH_X86 || KMP_ARCH_PPC64 || (KMP_OS_LINUX && KMP_ARCH_AARCH64)
+# if KMP_ARCH_X86
 kmp_int8
 __kmp_test_then_add8( volatile kmp_int8 *p, kmp_int8 d )
 {
@@ -462,7 +462,7 @@ __kmp_test_then_add64( volatile kmp_int6
     }
     return old_value;
 }
-# endif /* KMP_ARCH_X86 || KMP_ARCH_PPC64 || (KMP_OS_LINUX && KMP_ARCH_AARCH64) */
+# endif /* KMP_ARCH_X86 */
 
 kmp_int64
 __kmp_test_then_or64( volatile kmp_int64 *p, kmp_int64 d )




More information about the Openmp-commits mailing list