diff options
| author | Josh Bleecher Snyder <josharian@gmail.com> | 2017-04-22 17:35:28 -0700 |
|---|---|---|
| committer | Josh Bleecher Snyder <josharian@gmail.com> | 2017-04-23 12:11:42 +0000 |
| commit | d40bb738ff59bada723fe5f834d41531391b532a (patch) | |
| tree | 677e336d39393d0d693c739c9a850524a2cc3689 /src/cmd/compile/internal/types/sym.go | |
| parent | 90bd5eccb9ed0964ee9739b6da077a8b1470198b (diff) | |
| download | go-d40bb738ff59bada723fe5f834d41531391b532a.tar.gz go-d40bb738ff59bada723fe5f834d41531391b532a.zip | |
cmd/compile: remove types.Sym.lsym
It was just a cache, and the CL series yesterday
removed 40% of the calls to types.Linksym in make.bash.
Testing atop CL 40693 (backend concurrency)
indicates that removing it is actually a very minor
performance improvement.
Passes toolstash-check.
Change-Id: I97c2973036964acdd11b3cb842bc31f33ae60389
Reviewed-on: https://go-review.googlesource.com/41492
Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to 'src/cmd/compile/internal/types/sym.go')
| -rw-r--r-- | src/cmd/compile/internal/types/sym.go | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/cmd/compile/internal/types/sym.go b/src/cmd/compile/internal/types/sym.go index e316e99ce7..d9b6d92abe 100644 --- a/src/cmd/compile/internal/types/sym.go +++ b/src/cmd/compile/internal/types/sym.go @@ -31,7 +31,6 @@ type Sym struct { flags bitset8 Label *Node // corresponding label (ephemeral) Origpkg *Pkg // original package for . import - lsym *obj.LSym } const ( @@ -78,8 +77,5 @@ func (sym *Sym) Linksym() *obj.LSym { if sym == nil { return nil } - if sym.lsym == nil { - sym.lsym = Ctxt.Lookup(sym.LinksymName()) - } - return sym.lsym + return Ctxt.Lookup(sym.LinksymName()) } |