-
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: spurious nil check in len(slice) #7316
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 |
rsc
changed the title
cmd/gc: spurious nil check in len(slice)
cmd/compile: spurious nil check in len(slice)
Jun 8, 2015
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: