aboutsummaryrefslogtreecommitdiff
path: root/src/cmd/compile/internal/types/sym.go
diff options
context:
space:
mode:
authorCherry Zhang <cherryyz@google.com>2019-09-11 16:17:01 -0400
committerCherry Zhang <cherryyz@google.com>2019-10-02 19:07:17 +0000
commit53b7c18284a404de6ca814bc3313d980b8e5ecc3 (patch)
tree3ca98fd1688fb07d21d8681eba83ae956e468a20 /src/cmd/compile/internal/types/sym.go
parentcd75cf4bc02a741f7b59e30dd1170364000fd134 (diff)
downloadgo-53b7c18284a404de6ca814bc3313d980b8e5ecc3.tar.gz
go-53b7c18284a404de6ca814bc3313d980b8e5ecc3.zip
[dev.link] cmd/compile, cmd/asm: assign index to symbols
We are planning to use indices for symbol references, instead of symbol names. Here we assign indices to symbols defined in the package being compiled, and propagate the indices to the dependent packages in the export data. A symbol is referenced by a tuple, (package index, symbol index). Normally, for a given symbol, this index is unique, and the symbol index is globally consistent (but with exceptions, see below). The package index is local to a compilation. For example, when compiling the fmt package, fmt.Println gets assigned index 25, then all packages that reference fmt.Println will refer it as (X, 25) with some X. X is the index for the fmt package, which may differ in different compilations. There are some symbols that do not have clear package affiliation, such as dupOK symbols and linknamed symbols. We cannot give them globally consistent indices. We categorize them as non-package symbols, assign them with package index 1 and a symbol index that is only meaningful locally. Currently nothing will consume the indices. All this is behind a flag, -newobj. The flag needs to be set for all builds (-gcflags=all=-newobj -asmflags=all=-newobj), or none. Change-Id: I18e489c531e9a9fbc668519af92c6116b7308cab Reviewed-on: https://go-review.googlesource.com/c/go/+/196029 Reviewed-by: Than McIntosh <thanm@google.com>
Diffstat (limited to 'src/cmd/compile/internal/types/sym.go')
-rw-r--r--src/cmd/compile/internal/types/sym.go15
1 files changed, 12 insertions, 3 deletions
diff --git a/src/cmd/compile/internal/types/sym.go b/src/cmd/compile/internal/types/sym.go
index c9dd9f399e..d43efd3bd0 100644
--- a/src/cmd/compile/internal/types/sym.go
+++ b/src/cmd/compile/internal/types/sym.go
@@ -76,15 +76,24 @@ func (sym *Sym) LinksymName() string {
return sym.Pkg.Prefix + "." + sym.Name
}
-func (sym *Sym) Linksym() *obj.LSym {
+func (sym *Sym) Linksym() (r *obj.LSym) {
if sym == nil {
return nil
}
if sym.Func() {
// This is a function symbol. Mark it as "internal ABI".
- return Ctxt.LookupABI(sym.LinksymName(), obj.ABIInternal)
+ r = Ctxt.LookupABI(sym.LinksymName(), obj.ABIInternal)
+ } else {
+ r = Ctxt.Lookup(sym.LinksymName())
}
- return Ctxt.Lookup(sym.LinksymName())
+ if r.Pkg == "" {
+ if sym.Linkname != "" {
+ r.Pkg = "_"
+ } else {
+ r.Pkg = sym.Pkg.Prefix
+ }
+ }
+ return
}
// Less reports whether symbol a is ordered before symbol b.