-
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: internal compiler error: unhandled OCONV STRUCT -> STRUCT #17111
Labels
Milestone
Comments
Looking. Simpler form: package main
type I int
var (
i int
x = I(i)
e interface{} = x
) |
Yeah, thanks @dsnet! |
CL https://golang.org/cl/29213 mentions this issue. |
josharian
added a commit
to josharian/go
that referenced
this issue
Sep 15, 2016
staticassign unwraps all CONVNOPs. However, in the included test, we need the CONVNOP for everything to typecheck. Stop unwrapping unnecessarily. The code we generate for this example is suboptimal, but that's not new; see golang#17113. Fixes golang#17111. Change-Id: I29532787a074a6fe19a5cc53271eb9c84bf1b576
I confirm that this fixed the broken target. Thanks! |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Commit dfc56a4 caused a regression where the following code fails to compile:
On the commit in question, I see the following:
I expect the test to compile and pass.
/cc @josharian, @randall77, @mdempsky
The text was updated successfully, but these errors were encountered: