mirror of
https://github.com/golang/go.git
synced 2025-12-08 06:10:04 +00:00
sync/atomic: amend comments for Value.{Swap,CompareAndSwap}
Related to CL 241661, CL 403094.
Change-Id: I86877d9a013b05b1a97f9aa1333cd96ce98469cb
GitHub-Last-Rev: 7fedca5848
GitHub-Pull-Request: golang/go#76259
Reviewed-on: https://go-review.googlesource.com/c/go/+/719660
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Keith Randall <khr@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
Reviewed-by: Keith Randall <khr@google.com>
Reviewed-by: Junyang Shao <shaojunyang@google.com>
This commit is contained in:
parent
e0807ba470
commit
c3d1d42764
1 changed files with 2 additions and 4 deletions
|
|
@ -98,8 +98,7 @@ func (v *Value) Swap(new any) (old any) {
|
|||
if typ == nil {
|
||||
// Attempt to start first store.
|
||||
// Disable preemption so that other goroutines can use
|
||||
// active spin wait to wait for completion; and so that
|
||||
// GC does not see the fake type accidentally.
|
||||
// active spin wait to wait for completion.
|
||||
runtime_procPin()
|
||||
if !CompareAndSwapPointer(&vp.typ, nil, unsafe.Pointer(&firstStoreInProgress)) {
|
||||
runtime_procUnpin()
|
||||
|
|
@ -150,8 +149,7 @@ func (v *Value) CompareAndSwap(old, new any) (swapped bool) {
|
|||
}
|
||||
// Attempt to start first store.
|
||||
// Disable preemption so that other goroutines can use
|
||||
// active spin wait to wait for completion; and so that
|
||||
// GC does not see the fake type accidentally.
|
||||
// active spin wait to wait for completion.
|
||||
runtime_procPin()
|
||||
if !CompareAndSwapPointer(&vp.typ, nil, unsafe.Pointer(&firstStoreInProgress)) {
|
||||
runtime_procUnpin()
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue