From 8496c6cd77e2f5f105fc47315680174157d66647 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sun, 1 Jan 2012 22:44:01 -0500 Subject: [PATCH] Use 4-byte slock_t on both PPC and PPC64. Previously we defined slock_t as 8 bytes on PPC64, but the TAS assembly code uses word-wide operations regardless, so that the second word was just wasted space. There doesn't appear to be any performance benefit in adding the second word, so get rid of it to simplify the code. --- src/include/storage/s_lock.h | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/include/storage/s_lock.h b/src/include/storage/s_lock.h index cc67be81eb..2e5163eabf 100644 --- a/src/include/storage/s_lock.h +++ b/src/include/storage/s_lock.h @@ -354,13 +354,10 @@ tas(volatile slock_t *lock) #if defined(__ppc__) || defined(__powerpc__) || defined(__ppc64__) || defined(__powerpc64__) #define HAS_TEST_AND_SET -#if defined(__ppc64__) || defined(__powerpc64__) -typedef unsigned long slock_t; -#else typedef unsigned int slock_t; -#endif #define TAS(lock) tas(lock) + /* * NOTE: per the Enhanced PowerPC Architecture manual, v1.0 dated 7-May-2002, * an isync is a sufficient synchronization barrier after a lwarx/stwcx loop. -- 2.40.0