c874af597b
The Go runtime silently chose the gold linker on arm and aarch64 build targets to work around a binutils bug. That bug was from 2016, and has been fixed in binutils for a while. But the Go workaround was still in place. The workaround to choose gold was supposed to error out during the configure phase if gold wasn't found - but that doesn't happen. This leads to builds on aarch64 failing if attempting to compile without gold. Just removing the special condition in the Go source code: https://go-review.googlesource.com/c/go/+/391115/ works, but that patch is unlikely to be merged upstream. A patch fixing the build is more likely to be merged: https://go-review.googlesource.com/c/go/+/366279/ which updates the linker selection to use gold only if available. References to this bug in upstream Go and Binutils: https://github.com/golang/go/issues/22040 https://sourceware.org/bugzilla/show_bug.cgi?id=19962 Including the likely-to-be-merged patch here.
54 lines
2.0 KiB
Diff
54 lines
2.0 KiB
Diff
From 5ccf9f47bf4f5ba53e0ab7338a7fd4626714cfb2 Mon Sep 17 00:00:00 2001
|
|
From: Jeffery To <jeffery.to@gmail.com>
|
|
Date: Tue, 23 Nov 2021 15:05:37 +0800
|
|
Subject: [PATCH] cmd/link: use gold on ARM/ARM64 only if gold is available
|
|
|
|
COPY relocation handling on ARM/ARM64 has been fixed in recent versions
|
|
of the GNU linker. This switches to gold only if gold is available.
|
|
|
|
Fixes #22040.
|
|
---
|
|
src/cmd/link/internal/ld/lib.go | 19 +++++++------------
|
|
1 file changed, 7 insertions(+), 12 deletions(-)
|
|
|
|
diff --git a/src/cmd/link/internal/ld/lib.go b/src/cmd/link/internal/ld/lib.go
|
|
index 9e13db7b71..2b379259a2 100644
|
|
--- a/src/cmd/link/internal/ld/lib.go
|
|
+++ b/src/cmd/link/internal/ld/lib.go
|
|
@@ -1390,25 +1390,20 @@ func (ctxt *Link) hostlink() {
|
|
}
|
|
|
|
if ctxt.Arch.InFamily(sys.ARM, sys.ARM64) && buildcfg.GOOS == "linux" {
|
|
- // On ARM, the GNU linker will generate COPY relocations
|
|
- // even with -znocopyreloc set.
|
|
+ // On ARM, older versions of the GNU linker will generate
|
|
+ // COPY relocations even with -znocopyreloc set.
|
|
// https://sourceware.org/bugzilla/show_bug.cgi?id=19962
|
|
//
|
|
- // On ARM64, the GNU linker will fail instead of
|
|
- // generating COPY relocations.
|
|
+ // On ARM64, older versions of the GNU linker will fail
|
|
+ // instead of generating COPY relocations.
|
|
//
|
|
- // In both cases, switch to gold.
|
|
- altLinker = "gold"
|
|
-
|
|
- // If gold is not installed, gcc will silently switch
|
|
- // back to ld.bfd. So we parse the version information
|
|
- // and provide a useful error if gold is missing.
|
|
+ // In both cases, switch to gold if gold is available.
|
|
name, args := flagExtld[0], flagExtld[1:]
|
|
args = append(args, "-fuse-ld=gold", "-Wl,--version")
|
|
cmd := exec.Command(name, args...)
|
|
if out, err := cmd.CombinedOutput(); err == nil {
|
|
- if !bytes.Contains(out, []byte("GNU gold")) {
|
|
- log.Fatalf("ARM external linker must be gold (issue #15696), but is not: %s", out)
|
|
+ if bytes.Contains(out, []byte("GNU gold")) {
|
|
+ altLinker = "gold"
|
|
}
|
|
}
|
|
}
|
|
--
|
|
2.32.0
|
|
|