-
Notifications
You must be signed in to change notification settings - Fork 17.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cmd/compile, cmd/link: relocation target go.builtin.error.Error not defined #29304
Comments
This worked in Go 1.10 but was broken in Go 1.11. cmd/compile succeeds but cmd/link fails. Not sure whether cmd/compile output garbage, though.
/cc @griesemer @mdempsky @josharian @cherrymui @heschik @ianlancetaylor |
Probably warrants a backport too. |
git bisect points to 80bbad0 (cmd/compile: omit unnecessary interface method expression wrappers). |
Simpler reproducer: package main
func main() {
_ = error.Error(nil)
} |
Change https://golang.org/cl/154578 mentions this issue: |
@gopherbot, please backport to Go 1.11. |
Backport issue(s) opened: #29307 (for 1.11). Remember to create the cherry-pick CL(s) as soon as the patch is submitted to master, according to https://golang.org/wiki/MinorReleases. |
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (
go env
)?Repro on play.golang.org.
What did you do?
https://play.golang.org/p/6NvGso5fqRg
What did you expect to see?
Some understandable/user-friendly error message, pointing to line 9. (Or, actually, maybe this line is actually correct and should compile? Not 100% sure. With go1.9 it seemed to compile correctly and work!)
What did you see instead?
The text was updated successfully, but these errors were encountered: