Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(42)

Issue 13157043: cmd/ld: faster linking

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 2 months ago by dvyukov
Modified:
12 years, 11 months ago
Reviewers:
Visibility:
Public.

Description

cmd/ld: faster linking Replace symbol list with array. Introduce helper arrays. $ time go build cmd/go real            user            sys before: 0m2.692s        0m2.179s        0m0.398s 0m2.692s        0m2.178s        0m0.398s 0m2.699s        0m2.178s        0m0.394s after: 0m2.582s 0m2.071s 0m0.411s 0m2.579s 0m2.069s 0m0.401s 0m2.580s 0m2.069s 0m0.409s

Patch Set 1 #

Patch Set 2 : diff -r a21eec46edee https://dvyukov%40google.com@code.google.com/p/go/ #

Patch Set 3 : diff -r f795546f1c55 https://dvyukov%40google.com@code.google.com/p/go/ #

Patch Set 4 : diff -r f795546f1c55 https://dvyukov%40google.com@code.google.com/p/go/ #

Patch Set 5 : diff -r 6752a7aad603 https://dvyukov%40google.com@code.google.com/p/go/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+190 lines, -75 lines) Patch
M src/cmd/5l/l.h View 1 2 3 1 chunk +0 lines, -1 line 0 comments Download
M src/cmd/6l/l.h View 1 2 chunks +0 lines, -2 lines 0 comments Download
M src/cmd/8l/l.h View 1 2 3 2 chunks +0 lines, -2 lines 0 comments Download
M src/cmd/ld/data.c View 1 2 3 4 3 chunks +10 lines, -2 lines 0 comments Download
M src/cmd/ld/elf.c View 1 2 3 4 2 chunks +6 lines, -1 line 0 comments Download
M src/cmd/ld/go.c View 1 2 3 4 7 chunks +31 lines, -33 lines 0 comments Download
M src/cmd/ld/lib.h View 1 2 3 4 3 chunks +52 lines, -1 line 0 comments Download
M src/cmd/ld/lib.c View 1 2 3 4 8 chunks +53 lines, -8 lines 0 comments Download
M src/cmd/ld/macho.c View 1 2 3 1 chunk +3 lines, -1 line 0 comments Download
M src/cmd/ld/pe.c View 1 2 3 4 2 chunks +4 lines, -2 lines 0 comments Download
M src/cmd/ld/symtab.c View 1 4 chunks +31 lines, -22 lines 0 comments Download

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b