Commit6249971

committed

Just noticed that use of 'volatile' in HPPA S_UNLOCK() was causing gcc

to generate unnecessarily stupid code. Tweak macro to describe a seriesof store-constant ops, not store/load/store/load/store/load/store.

1 parent810c0b8 commit6249971Copy full SHA for 6249971

File tree

-3

lines changed

-3

lines changed

Lines changed: 5 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`*`
`10`	`10`	`*`
`11`	`11`	`* IDENTIFICATION`
`12`		`- * $Header: /cvsroot/pgsql/src/include/storage/s_lock.h,v 1.88 2001/02/10 04:07:25 tgl Exp $`
	`12`	`+ * $Header: /cvsroot/pgsql/src/include/storage/s_lock.h,v 1.89 2001/02/16 23:50:40 tgl Exp $`
`13`	`13`	`*`
`14`	`14`	`*-------------------------------------------------------------------------`
`15`	`15`	`*/`
`@@ -392,8 +392,10 @@ tas(volatile slock_t *lock)`
`392`	`392`	`#defineS_UNLOCK(lock) \`
`393`	`393`	`do { \`
`394`	`394`	`volatile slock_t lock_ = (volatile slock_t ) (lock); \`
`395`		`-lock_->sema[0] = lock_->sema[1] = \`
`396`		`-lock_->sema[2] = lock_->sema[3] = -1; \`
	`395`	`+lock_->sema[0] = -1; \`
	`396`	`+lock_->sema[1] = -1; \`
	`397`	`+lock_->sema[2] = -1; \`
	`398`	`+lock_->sema[3] = -1; \`
`397`	`399`	`} while (0)`
`398`	`400`
`399`	`401`	`#defineS_LOCK_FREE(lock)( (int ) (((long) (lock) + 15) & ~15) != 0)`

Comments

(0)