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: ICE: not lowered: v58, ITab PTR64 PTR64 #22327
Comments
go1.2 reports I think go1.2 is correct. I think if we reported that error correctly, we'd never pass the bad code to the backend, and then we wouldn't ICE. |
go1.4 generated code is "correct", in that it does convert the *int to an interface{} and appends it to a []interface{}. |
Hmmm, looks like |
Oh, yes, the code should compile correctly, like it did with go1.4. go/types and gccgo accept the code. (Sorry, I should have mentioned that in the original report.) |
Looking at cmd/compile's -W output, I think maybe this is an issue in walk instead. Currently we're rewriting
I think what we actually mean to do is rewrite it to:
|
Still happening with |
This has been broken for a while, so punting to 1.12, but marking as a release-blocker for that release. |
Looks like there is a missing
After order:
Nowhere is there a |
Change https://golang.org/cl/142718 mentions this issue: |
At tip, this package fails to compile with an ICE in SSA:
/cc @randall77
The text was updated successfully, but these errors were encountered: