diff options
author | Cherry Zhang <cherryyz@google.com> | 2016-09-06 08:48:14 -0400 |
---|---|---|
committer | Cherry Zhang <cherryyz@google.com> | 2016-09-06 14:44:50 +0000 |
commit | 644c16c76cc11034b124763a039a7164cb3e1abc (patch) | |
tree | 038bc78b57874ec2183b2da17e33a0cb91a35a8e /test/intrinsic.dir | |
parent | f1ef5a06d296fd0ba604a62ebb30531cef7ae74a (diff) | |
download | go-644c16c76cc11034b124763a039a7164cb3e1abc.tar.gz go-644c16c76cc11034b124763a039a7164cb3e1abc.zip |
cmd/compile: fix intrinsifying sync/atomic.Swap* on AMD64
It should alias to Xchg instead of Swap. Found when testing #16985.
Change-Id: If9fd734a1f89b8b2656f421eb31b9d1b0d95a49f
Reviewed-on: https://go-review.googlesource.com/28512
Run-TryBot: Cherry Zhang <cherryyz@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
Diffstat (limited to 'test/intrinsic.dir')
-rw-r--r-- | test/intrinsic.dir/atomic.go | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/test/intrinsic.dir/atomic.go b/test/intrinsic.dir/atomic.go new file mode 100644 index 0000000000..71468a1b38 --- /dev/null +++ b/test/intrinsic.dir/atomic.go @@ -0,0 +1,19 @@ +// +build amd64 + +// Copyright 2016 The Go Authors. All rights reserved. +// Use of this source code is governed by a BSD-style +// license that can be found in the LICENSE file. + +package main + +import "sync/atomic" + +var x uint32 + +func atomics() { + _ = atomic.LoadUint32(&x) // ERROR "intrinsic substitution for LoadUint32" + atomic.StoreUint32(&x, 1) // ERROR "intrinsic substitution for StoreUint32" + atomic.AddUint32(&x, 1) // ERROR "intrinsic substitution for AddUint32" + atomic.SwapUint32(&x, 1) // ERROR "intrinsic substitution for SwapUint32" + atomic.CompareAndSwapUint32(&x, 1, 2) // ERROR "intrinsic substitution for CompareAndSwapUint32" +} |