-
Notifications
You must be signed in to change notification settings - Fork 18k
cmd/compile: spurious nil check in len(slice) #7316
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
Milestone
Comments
Should understand these inefficiencies for Go 1.3. Labels changed: added release-go1.3. Owner changed to @rsc. Status changed to Accepted. |
CL https://golang.org/cl/96720043 mentions this issue. |
CL https://golang.org/cl/91850043 mentions this issue. |
This issue was updated by revision 03c0f3f. LGTM=rsc R=rsc, daniel.morsing, bradfitz CC=dave, golang-codereviews https://golang.org/cl/91850043 Committer: Russ Cox |
go1.8 gives (morestack excluded):
The spurious nilcheck mentioned in the title is no longer there, but maybe @josharian can decide if the registerization is now good enough (and we can close this) or it needs to stay open. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The text was updated successfully, but these errors were encountered: