From 6249971b41470f054ffd1525cde2180cea19087f Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Fri, 16 Feb 2001 23:50:40 +0000 Subject: [PATCH] Just noticed that use of 'volatile' in HPPA S_UNLOCK() was causing gcc to generate unnecessarily stupid code. Tweak macro to describe a series of store-constant ops, not store/load/store/load/store/load/store. --- src/include/storage/s_lock.h | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/include/storage/s_lock.h b/src/include/storage/s_lock.h index 726c3645e5..9cf73163d6 100644 --- a/src/include/storage/s_lock.h +++ b/src/include/storage/s_lock.h @@ -9,7 +9,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/include/storage/s_lock.h,v 1.88 2001/02/10 04:07:25 tgl Exp $ + * $Header: /cvsroot/pgsql/src/include/storage/s_lock.h,v 1.89 2001/02/16 23:50:40 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -392,8 +392,10 @@ tas(volatile slock_t *lock) #define S_UNLOCK(lock) \ do { \ volatile slock_t *lock_ = (volatile slock_t *) (lock); \ - lock_->sema[0] = lock_->sema[1] = \ - lock_->sema[2] = lock_->sema[3] = -1; \ + lock_->sema[0] = -1; \ + lock_->sema[1] = -1; \ + lock_->sema[2] = -1; \ + lock_->sema[3] = -1; \ } while (0) #define S_LOCK_FREE(lock) ( *(int *) (((long) (lock) + 15) & ~15) != 0) -- 2.40.0