diff options
| author | Keith Randall <khr@google.com> | 2018-11-26 10:48:56 -0800 |
|---|---|---|
| committer | Keith Randall <khr@golang.org> | 2018-11-26 20:00:59 +0000 |
| commit | ca3749230b5a7d43b3292226fdb2b6f3de5d653b (patch) | |
| tree | 228f42320e098e6cc90844b87a566523b8c35897 /src/runtime/testdata | |
| parent | 9ab2ffe8e92f9660cbde1a18921ae864c64f280b (diff) | |
| download | go-ca3749230b5a7d43b3292226fdb2b6f3de5d653b.tar.xz | |
cmd/compile: allow bodyless function if it is linkname'd
In assembly free packages (aka "complete" or "pure go"), allow
bodyless functions if they are linkname'd to something else.
Presumably the thing the function is linkname'd to has a definition.
If not, the linker will complain. And linkname is unsafe, so we expect
users to know what they are doing.
Note this handles only one direction, where the linkname directive
is in the local package. If the linkname directive is in the remote
package, this CL won't help. (See os/signal/sig.s for an example.)
Fixes #23311
Change-Id: I824361b4b582ee05976d94812e5b0e8b0f7a18a6
Reviewed-on: https://go-review.googlesource.com/c/151318
Run-TryBot: Keith Randall <khr@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
Diffstat (limited to 'src/runtime/testdata')
| -rw-r--r-- | src/runtime/testdata/testprog/empty.s | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/runtime/testdata/testprog/empty.s b/src/runtime/testdata/testprog/empty.s deleted file mode 100644 index c5aa6f8a54..0000000000 --- a/src/runtime/testdata/testprog/empty.s +++ /dev/null @@ -1,5 +0,0 @@ -// Copyright 2018 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. - -// This exists solely so we can linkname in symbols from runtime. |