-
Notifications
You must be signed in to change notification settings - Fork 18k
cmd/compile: internal compiler error: Int64Val(10000000000000000000) (dev.regabi branch) #43480
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
Labels
Comments
Bisects to CL 272654 (unsurprising, I guess). |
Change https://golang.org/cl/281272 mentions this issue: |
This comment has been minimized.
This comment has been minimized.
gopherbot
pushed a commit
that referenced
this issue
Jan 4, 2021
It's an error to call Int64Val on constants that don't fit into int64. CL 272654 made the compiler stricter about detecting misuse, and revealed that we were using it improperly in detecting consecutive integer-switch cases. That particular usage actually did work in practice, but it's easy and best to just fix it. Fixes #43480. Change-Id: I56f722d75e83091638ac43b80e45df0b0ad7d48d Reviewed-on: https://go-review.googlesource.com/c/go/+/281272 Trust: Matthew Dempsky <[email protected]> Run-TryBot: Matthew Dempsky <[email protected]> TryBot-Result: Go Bot <[email protected]> Reviewed-by: Cuong Manh Le <[email protected]>
Closing as fixed on |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
No, only on the regabi branch.
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
Cloned https://github.com/ericlagergren/decimal, built the dev.regabi branch, and did a
go build
in thedecimal
checkout.What did you expect to see?
No errors.
What did you see instead?
/cc @mdempsky
The text was updated successfully, but these errors were encountered: