]> Cypherpunks repositories - gostls13.git/commit
cmd/internal/obj, cmd/link: put all symbol data in one contiguous section
authorMichael Hudson-Doyle <michael.hudson@canonical.com>
Sun, 20 Mar 2016 21:55:20 +0000 (10:55 +1300)
committerMichael Hudson-Doyle <michael.hudson@canonical.com>
Mon, 21 Mar 2016 01:36:00 +0000 (01:36 +0000)
commit36d5650a1c60be5e09abc89774758f525afc7874
tree1ea3427f5c56f804d2c15b370506ece32209876c
parent34f0c0b3de8957f247c0bc99a682f622793fd88b
cmd/internal/obj, cmd/link: put all symbol data in one contiguous section

Another object file change, gives a reasonable improvement:

name       old s/op   new s/op   delta
LinkCmdGo  0.46 ± 3%  0.44 ± 9%  -3.34%  (p=0.000 n=98+82)
LinkJuju   4.09 ± 4%  3.92 ± 5%  -4.30%  (p=0.000 n=98+99)

I guess the data section could be mmap-ed instead of read, I haven't tried
that.

Change-Id: I959eee470a05526ab1579e3f5d3ede41c16c954f
Reviewed-on: https://go-review.googlesource.com/20928
Run-TryBot: Michael Hudson-Doyle <michael.hudson@canonical.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: David Crawshaw <crawshaw@golang.org>
src/cmd/internal/goobj/read.go
src/cmd/internal/obj/objfile.go
src/cmd/link/internal/ld/objfile.go