Toshaan Bharvani
2 years ago
4 changed files with 76 additions and 10 deletions
@ -0,0 +1,53 @@ |
|||||||
|
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 |
||||||
|
|
@ -0,0 +1,7 @@ |
|||||||
|
// +build rpm_crashtraceback |
||||||
|
|
||||||
|
package runtime |
||||||
|
|
||||||
|
func init() { |
||||||
|
setTraceback("crash") |
||||||
|
} |
@ -1 +1 @@ |
|||||||
add-auto-load-safe-path /usr/lib/golang/src/pkg/runtime/runtime-gdb.py |
add-auto-load-safe-path /usr/lib/golang/src/runtime/runtime-gdb.py |
||||||
|
Loading…
Reference in new issue