]> Cypherpunks repositories - gostls13.git/commit
cmd/go, cmd/link: provide meaningful error msg with ext linking on ppc64
authorLynn Boger <laboger@linux.vnet.ibm.com>
Fri, 27 Apr 2018 12:07:48 +0000 (08:07 -0400)
committerBrad Fitzpatrick <bradfitz@golang.org>
Fri, 4 May 2018 15:05:41 +0000 (15:05 +0000)
commit506d6a32cecfed3f3b310f9b0d3de10205c17852
tree3945e403890e11256cddab5ba9093bb8f3cda7f9
parent166c37a7d0f2753be9fc5b16a9ab65e9e3d6975c
cmd/go, cmd/link: provide meaningful error msg with ext linking on ppc64

linux/ppc64 uses the ppc64 v1 ABI which was never fully supported
by Go. (linux/ppc64le uses the ppc64 v2 ABI and that is fully
supported).

As a result if the external linker is used to build a program
on ppc64, there is a either a warning or error message that doesn't
clearly describe the problem. In the case of a warning,
a program is created that will most likely not execute since it is not
built as expected for the ppc64 dynamic linker (ld64.so.1).

To avoid confusion in these cases, error messages are now issued
if external linker is explicitly used to build the program. Note that most
buildmodes that require external linking were already flagging linux/ppc64
as unsupported except for c-archive, which has been added here.

This problem does not occur with gccgo since the ppc64 v1 ABI is
supported there.

Fixes #25079

Change-Id: I44d77a1eb9df750d499cd432b0ca4a97f0be88b2
Reviewed-on: https://go-review.googlesource.com/109915
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/cmd/go/internal/work/init.go
src/cmd/link/internal/ld/config.go