]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/dist: skip internal linking tests on arm64
authorBenny Siegert <bsiegert@gmail.com>
Sat, 5 Feb 2022 16:23:26 +0000 (17:23 +0100)
committerBenny Siegert <bsiegert@gmail.com>
Mon, 7 Feb 2022 06:38:18 +0000 (06:38 +0000)
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 <bsiegert@gmail.com>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
src/cmd/dist/test.go

index 50a2e5936c639846d8d9f78f78756552fcf10390..4b67565430cc0608f529c4dad690436f1e384154 100644 (file)
@@ -1117,9 +1117,9 @@ func (t *tester) cgoTest(dt *distTest) error {
        cmd := t.addCmd(dt, "misc/cgo/test", t.goTest())
        setEnv(cmd, "GOFLAGS", "-ldflags=-linkmode=auto")
 
-       // Skip internal linking cases on linux/arm64 to support GCC-9.4 and above.
+       // Skip internal linking cases on arm64 to support GCC-9.4 and above.
        // See issue #39466.
-       skipInternalLink := goarch == "arm64" && goos == "linux"
+       skipInternalLink := goarch == "arm64" && goos != "windows"
 
        if t.internalLink() && !skipInternalLink {
                cmd := t.addCmd(dt, "misc/cgo/test", t.goTest(), "-tags=internal")