From 06da8fd84afc4694d4dbab9316ae701bd4dfaa8d Mon Sep 17 00:00:00 2001 From: Michael Hudson-Doyle Date: Thu, 3 Sep 2015 09:05:25 +1200 Subject: [PATCH] cmd/link: only embed runtime.goarm in the module that contains the runtime package And do it properly so freebsd and nacl still work. Change-Id: I6f9f30e93ceae6dee59215ed608c6a158bdbdbb0 Reviewed-on: https://go-review.googlesource.com/14280 Run-TryBot: Michael Hudson-Doyle TryBot-Result: Gobot Gobot Reviewed-by: Ian Lance Taylor --- src/cmd/link/internal/arm/obj.go | 6 ------ src/cmd/link/internal/ld/lib.go | 32 ++++++++++++++++++++++---------- 2 files changed, 22 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..89f805d483 100644 --- a/src/cmd/link/internal/ld/lib.go +++ b/src/cmd/link/internal/ld/lib.go @@ -558,19 +558,31 @@ 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 + s.Size = 0 + 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 -- 2.48.1