diff options
| author | Michael Hudson-Doyle <michael.hudson@canonical.com> | 2015-09-03 09:05:25 +1200 |
|---|---|---|
| committer | David Crawshaw <crawshaw@golang.org> | 2015-09-03 16:22:21 +0000 |
| commit | bf99d8f843ae3dfa7a3a4cd5c17aec79e2b9997f (patch) | |
| tree | 864ce5c7e1f473374366fc8324acd13aa022cef6 /src | |
| parent | 9e6ba37b86d0390ac1c5eb80731ca131b6801b7a (diff) | |
| download | go-bf99d8f843ae3dfa7a3a4cd5c17aec79e2b9997f.tar.xz | |
cmd/link: only embed runtime.goarm in the module that contains the runtime package
Change-Id: Ia18984343ca4ced3671d967ff9a5b0e32874430c
Reviewed-on: https://go-review.googlesource.com/14220
Reviewed-by: David Crawshaw <crawshaw@golang.org>
Run-TryBot: David Crawshaw <crawshaw@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/cmd/link/internal/arm/obj.go | 6 | ||||
| -rw-r--r-- | src/cmd/link/internal/ld/lib.go | 31 |
2 files changed, 21 insertions, 16 deletions
diff --git a/src/cmd/link/internal/arm/obj.go b/src/cmd/link/internal/arm/obj.go index 14fe7a64eb..c4678209ce 100644 --- a/src/cmd/link/internal/arm/obj.go +++ b/src/cmd/link/internal/arm/obj.go @@ -169,10 +169,4 @@ func archinit() { if ld.INITDAT != 0 && ld.INITRND != 0 { fmt.Printf("warning: -D0x%x is ignored because of -R0x%x\n", uint64(ld.INITDAT), uint32(ld.INITRND)) } - - // embed goarm to runtime.goarm - s := ld.Linklookup(ld.Ctxt, "runtime.goarm", 0) - - s.Type = obj.SRODATA - ld.Adduint8(ld.Ctxt, s, uint8(ld.Ctxt.Goarm)) } diff --git a/src/cmd/link/internal/ld/lib.go b/src/cmd/link/internal/ld/lib.go index 66f0a514a4..ae56a653dd 100644 --- a/src/cmd/link/internal/ld/lib.go +++ b/src/cmd/link/internal/ld/lib.go @@ -558,19 +558,30 @@ func loadlib() { Ctxt.Tlsg = tlsg moduledata := Linklookup(Ctxt, "runtime.firstmoduledata", 0) - if moduledata.Type == 0 || moduledata.Type == obj.SDYNIMPORT { - // If the module we are linking does not define the - // runtime.firstmoduledata symbol, create a local symbol for - // the moduledata. + if moduledata.Type != 0 && moduledata.Type != obj.SDYNIMPORT { + // If the module (toolchain-speak for "executable or shared + // library") we are linking contains the runtime package, it + // will define the runtime.firstmoduledata symbol and we + // truncate it back to 0 bytes so we can define its entire + // contents in symtab.go:symtab(). + moduledata.Size = 0 + + // In addition, on ARM, the runtime depends on the linker + // recording the value of GOARM. + if Thearch.Thechar == '5' { + s := Linklookup(Ctxt, "runtime.goarm", 0) + + s.Type = obj.SRODATA + Adduint8(Ctxt, s, uint8(Ctxt.Goarm)) + } + } else { + // If OTOH the module does not contain the runtime package, + // create a local symbol for the moduledata. moduledata = Linklookup(Ctxt, "local.moduledata", 0) moduledata.Local = true - } else { - // If OTOH the module does define the symbol, we truncate the - // symbol back to 0 bytes so we can define its entire - // contents. - moduledata.Size = 0 } - // Either way we mark it as noptrdata to hide it from the GC. + // In all cases way we mark the moduledata as noptrdata to hide it from + // the GC. moduledata.Type = obj.SNOPTRDATA moduledata.Reachable = true Ctxt.Moduledata = moduledata |