]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/compile: drop SSA atomics TODO
authorMichael Pratt <mpratt@google.com>
Fri, 9 Oct 2020 16:44:07 +0000 (12:44 -0400)
committerMichael Pratt <mpratt@google.com>
Fri, 9 Oct 2020 18:49:12 +0000 (18:49 +0000)
These ops have been used for sync/atomic since golang.org/cl/28076.

Change-Id: Ic1445c073273f6b191b97018e8eb6f7ad9c48922
Reviewed-on: https://go-review.googlesource.com/c/go/+/261077
Trust: Michael Pratt <mpratt@google.com>
Run-TryBot: Michael Pratt <mpratt@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
src/cmd/compile/internal/ssa/gen/genericOps.go

index 3518dd1e3cd4e9904a1049621ac536b2111f8517..85839303c59f5debe6f706d7908db5da8f51885b 100644 (file)
@@ -541,10 +541,10 @@ var genericOps = []opData{
        {name: "SelectN", argLength: 1, aux: "Int64"},     // arg0=tuple, auxint=field index.  Returns the auxint'th member.
        {name: "SelectNAddr", argLength: 1, aux: "Int64"}, // arg0=tuple, auxint=field index.  Returns the address of auxint'th member. Used for un-SSA-able result types.
 
-       // Atomic operations used for semantically inlining runtime/internal/atomic.
-       // Atomic loads return a new memory so that the loads are properly ordered
-       // with respect to other loads and stores.
-       // TODO: use for sync/atomic at some point.
+       // Atomic operations used for semantically inlining sync/atomic and
+       // runtime/internal/atomic. Atomic loads return a new memory so that
+       // the loads are properly ordered with respect to other loads and
+       // stores.
        {name: "AtomicLoad8", argLength: 2, typ: "(UInt8,Mem)"},                                    // Load from arg0.  arg1=memory.  Returns loaded value and new memory.
        {name: "AtomicLoad32", argLength: 2, typ: "(UInt32,Mem)"},                                  // Load from arg0.  arg1=memory.  Returns loaded value and new memory.
        {name: "AtomicLoad64", argLength: 2, typ: "(UInt64,Mem)"},                                  // Load from arg0.  arg1=memory.  Returns loaded value and new memory.