Skip to content
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.

Sign up for GitHub

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

Jump to bottom

cmd/compile: compiler array indexing improvements #2537

Closed
rsc opened this issue Dec 7, 2011 · 9 comments
Closed

cmd/compile: compiler array indexing improvements #2537

rsc opened this issue Dec 7, 2011 · 9 comments
Labels
FrozenDueToAge
Milestone
Unplanned

Comments

@rsc
Copy link
Contributor

rsc commented Dec 7, 2011 

CL 5449077 should not be necessary.

var x [4][256]byte
_ = x[1][i]

should be just as efficient as

var x0, x1, x2, x3 [256]byte
_ = x1[i]
@rsc
Copy link
Contributor Author

rsc commented Dec 9, 2011

Comment 1:

Labels changed: added priority-someday, removed priority-low.

@rsc
Copy link
Contributor Author

rsc commented Sep 12, 2012

Comment 3:

Labels changed: added go1.1maybe.

@robpike
Copy link
Contributor

robpike commented Mar 7, 2013

Comment 4:

Labels changed: removed go1.1maybe.

@rsc
Copy link
Contributor Author

rsc commented Jul 30, 2013

Comment 5:

Labels changed: added go1.3.

@robpike
Copy link
Contributor

robpike commented Aug 20, 2013

Comment 6:

Labels changed: removed go1.3.

@rsc
Copy link
Contributor Author

rsc commented Dec 4, 2013

Comment 7:

Labels changed: added repo-main.

@rsc
Copy link
Contributor Author

rsc commented Mar 3, 2014

Comment 8:

Adding Release=None to all Priority=Someday bugs.

Labels changed: added release-none.

@rsc rsc self-assigned this Mar 3, 2014
@rsc rsc added this to the Unplanned milestone Apr 10, 2015
@rsc rsc changed the title cmd/gc: compiler array indexing improvements cmd/compile: compiler array indexing improvements Jun 8, 2015
@dgryski
Copy link
Contributor

dgryski commented Jan 29, 2018

Is this still valid with the new SSA backend?

https://godbolt.org/g/h6w53Q seems to suggest it's generating the same code.

@randall77
Copy link
Contributor

Yes, I think this is fixed.

@golang golang locked and limited conversation to collaborators Jan 29, 2019
@rsc rsc removed their assignment Jun 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
Unplanned
Development

No branches or pull requests
5 participants
@rsc @dgryski @robpike @randall77 @gopherbot