]> Cypherpunks repositories - gostls13.git/commit
[dev.link] cmd/link: tweaks to data alignment processing
authorThan McIntosh <thanm@google.com>
Thu, 30 Apr 2020 20:07:31 +0000 (16:07 -0400)
committerThan McIntosh <thanm@google.com>
Fri, 1 May 2020 01:51:18 +0000 (01:51 +0000)
commitf875f8fe76160699432cdab8a23cfd292462c2b3
tree21a257f862c2ae9210dea7ba28e605aa2fca9399
parent2f83d68926e07109f8a32f7b01209ebebf4512f8
[dev.link] cmd/link: tweaks to data alignment processing

Now that the loader's internal storage mechanism for symbol alignment
is array-based and not map-based, we can go back to computing symbol
alignment in the parallel-by-section section of dodata.

With this patch plus the previous one, this produces a small
kubelet speedup:

$ benchstat out.devlink.txt out.align.txt
name                        old time/op  new time/op  delta
RelinkKubelet                13.3s ± 2%   13.1s ± 2%  -1.27%  (p=0.000 n=20+20)
RelinkKubelet-WithoutDebug   7.36s ± 5%   7.14s ± 3%  -3.00%  (p=0.000 n=20+20)

Change-Id: I9eb0e8fea6aeb12f188f499e9031d5a3a23232c7
Reviewed-on: https://go-review.googlesource.com/c/go/+/231221
Run-TryBot: Than McIntosh <thanm@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Reviewed-by: Jeremy Faller <jeremy@golang.org>
src/cmd/link/internal/ld/data.go