aboutsummaryrefslogtreecommitdiff
path: root/src/runtime/sys_darwin.go
diff options
context:
space:
mode:
authorAustin Clements <austin@google.com>2019-05-31 16:38:56 -0400
committerAustin Clements <austin@google.com>2019-06-06 19:44:12 +0000
commitf5e5bc1a4281b22c9ca2c75431647638de515ba0 (patch)
tree72e401c8b96177f5eef8d30a676e0779e15b9eab /src/runtime/sys_darwin.go
parentdde7c770ef44d45b8a9f98bf0f46556f18df2f6e (diff)
downloadgo-f5e5bc1a4281b22c9ca2c75431647638de515ba0.tar.gz
go-f5e5bc1a4281b22c9ca2c75431647638de515ba0.zip
runtime: mark all Go symbols called from assembly in other packages
This marks all Go symbols called from assembly in other packages with "go:linkname" directives to ensure they get ABI wrappers. Now that we have this go:linkname convention, this also removes the abi0Syms definition in the runtime, which was used to give morestackc an ABI0 wrapper. Instead, we now just mark morestackc with a go:linkname directive. This was tested with buildall.bash in the default configuration, with -race, and with -gcflags=all=-d=ssa/intrinsics/off. Since I couldn't test cgo on non-Linux configurations, I manually grepped for runtime symbols in runtime/cgo. Updates #31230. Change-Id: I6c8aa56be2ca6802dfa2bf159e49c411b9071bf1 Reviewed-on: https://go-review.googlesource.com/c/go/+/179862 Run-TryBot: Austin Clements <austin@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Michael Knyszek <mknyszek@google.com>
Diffstat (limited to 'src/runtime/sys_darwin.go')
-rw-r--r--src/runtime/sys_darwin.go3
1 files changed, 3 insertions, 0 deletions
diff --git a/src/runtime/sys_darwin.go b/src/runtime/sys_darwin.go
index 0bf17c47af..b50d441d92 100644
--- a/src/runtime/sys_darwin.go
+++ b/src/runtime/sys_darwin.go
@@ -206,6 +206,9 @@ func close_trampoline()
//go:nosplit
//go:cgo_unsafe_args
+//
+// This is exported via linkname to assembly in runtime/cgo.
+//go:linkname exit
func exit(code int32) {
libcCall(unsafe.Pointer(funcPC(exit_trampoline)), unsafe.Pointer(&code))
}