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/link: move away from *LSym linked lists #15374

Open
crawshaw opened this Issue Apr 19, 2016 · 2 comments

Comments

Projects
None yet
3 participants
@crawshaw
Contributor

crawshaw commented Apr 19, 2016

The linked lists come from the original C code. Slices make the code a bit easier to follow.

@crawshaw crawshaw added this to the Unreleased milestone Apr 19, 2016

@gopherbot

This comment has been minimized.

gopherbot commented Apr 19, 2016

CL https://golang.org/cl/22243 mentions this issue.

@gopherbot

This comment has been minimized.

gopherbot commented Apr 19, 2016

CL https://golang.org/cl/22205 mentions this issue.

gopherbot pushed a commit that referenced this issue Apr 20, 2016

cmd/link: process data symbols with slices
First (and largest single) step to switching cmd/link from linked
lists of symbols to slices.

Sort sections independently and concurrently.
This reduces jujud link times on linux/amd64 by ~4%.

Updates #15374

Change-Id: I452bc8f33081039468636502fe3c1cc8d6ed9efa
Reviewed-on: https://go-review.googlesource.com/22205
Reviewed-by: Michael Hudson-Doyle <michael.hudson@canonical.com>

gopherbot pushed a commit that referenced this issue Apr 21, 2016

cmd/link: convert textp into a slice
Updates #15374

Change-Id: I3ea715735862fe9550b88d7a29def6cb9d4419a6
Reviewed-on: https://go-review.googlesource.com/22243
Run-TryBot: David Crawshaw <crawshaw@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Michael Hudson-Doyle <michael.hudson@canonical.com>

@ALTree ALTree modified the milestones: Unplanned, Unreleased Jul 18, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment