Skip to content

Commit 1d60513

Browse files
committed
cmd/dist: skip internal linking tests on arm64
The previous workaround for issue #39466 only disabled this test for Linux. However, the issue manifests for all arm64 systems with gcc 9.4 and above. The new netbsd-arm64 builder uses NetBSD-current with gcc 10.3, so it fails in the same way. Updates #39466 Change-Id: I276a99a5e60914e5c22f74a680e461bea17cfe92 Reviewed-on: https://go-review.googlesource.com/c/go/+/383554 Trust: Benny Siegert <[email protected]> Run-TryBot: Ian Lance Taylor <[email protected]> TryBot-Result: Gopher Robot <[email protected]> Reviewed-by: Ian Lance Taylor <[email protected]>
1 parent d588f48 commit 1d60513

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/cmd/dist/test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1117,9 +1117,9 @@ func (t *tester) cgoTest(dt *distTest) error {
11171117
cmd := t.addCmd(dt, "misc/cgo/test", t.goTest())
11181118
setEnv(cmd, "GOFLAGS", "-ldflags=-linkmode=auto")
11191119

1120-
// Skip internal linking cases on linux/arm64 to support GCC-9.4 and above.
1120+
// Skip internal linking cases on arm64 to support GCC-9.4 and above.
11211121
// See issue #39466.
1122-
skipInternalLink := goarch == "arm64" && goos == "linux"
1122+
skipInternalLink := goarch == "arm64" && goos != "windows"
11231123

11241124
if t.internalLink() && !skipInternalLink {
11251125
cmd := t.addCmd(dt, "misc/cgo/test", t.goTest(), "-tags=internal")

0 commit comments

Comments
 (0)