diff options
author | Austin Clements <austin@google.com> | 2020-04-03 12:22:27 -0400 |
---|---|---|
committer | Austin Clements <austin@google.com> | 2020-04-08 18:35:49 +0000 |
commit | f7e6ab44b410ef56bb89da182948a451d3cca34c (patch) | |
tree | 9d49b461aff716c2d398e720c9bee93c20e1898e /misc | |
parent | da8591b61c141ca58e4b3eae382d2827006344fd (diff) | |
download | go-f7e6ab44b410ef56bb89da182948a451d3cca34c.tar.gz go-f7e6ab44b410ef56bb89da182948a451d3cca34c.zip |
all: remove scattered remnants of darwin/arm
This removes all conditions and conditional code (that I could find)
that depended on darwin/arm.
Fixes #35439 (since that only happened on darwin/arm)
Fixes #37611.
Change-Id: Ia4c32a5a4368ed75231075832b0b5bfb1ad11986
Reviewed-on: https://go-review.googlesource.com/c/go/+/227198
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Diffstat (limited to 'misc')
-rw-r--r-- | misc/cgo/test/pkg_test.go | 2 | ||||
-rw-r--r-- | misc/cgo/test/sigaltstack.go | 2 | ||||
-rw-r--r-- | misc/cgo/test/testx.go | 2 | ||||
-rw-r--r-- | misc/cgo/testcarchive/carchive_test.go | 8 | ||||
-rw-r--r-- | misc/cgo/testcshared/cshared_test.go | 2 | ||||
-rw-r--r-- | misc/cgo/testso/so_test.go | 2 | ||||
-rw-r--r-- | misc/cgo/testsovar/so_test.go | 2 | ||||
-rwxr-xr-x | misc/ios/clangwrap.sh | 4 |
8 files changed, 11 insertions, 13 deletions
diff --git a/misc/cgo/test/pkg_test.go b/misc/cgo/test/pkg_test.go index 08e075c022..26c50ad883 100644 --- a/misc/cgo/test/pkg_test.go +++ b/misc/cgo/test/pkg_test.go @@ -32,7 +32,7 @@ func TestCrossPackageTests(t *testing.T) { t.Skip("Can't exec cmd/go subprocess on Android.") case "darwin": switch runtime.GOARCH { - case "arm", "arm64": + case "arm64": t.Skip("Can't exec cmd/go subprocess on iOS.") } } diff --git a/misc/cgo/test/sigaltstack.go b/misc/cgo/test/sigaltstack.go index 7b3f4acbb7..f501f9ba3a 100644 --- a/misc/cgo/test/sigaltstack.go +++ b/misc/cgo/test/sigaltstack.go @@ -62,7 +62,7 @@ import ( func testSigaltstack(t *testing.T) { switch { - case runtime.GOOS == "solaris", runtime.GOOS == "illumos", runtime.GOOS == "darwin" && (runtime.GOARCH == "arm" || runtime.GOARCH == "arm64"): + case runtime.GOOS == "solaris", runtime.GOOS == "illumos", runtime.GOOS == "darwin" && runtime.GOARCH == "arm64": t.Skipf("switching signal stack not implemented on %s/%s", runtime.GOOS, runtime.GOARCH) case runtime.GOOS == "darwin" && runtime.GOARCH == "386": t.Skipf("sigaltstack fails on darwin/386") diff --git a/misc/cgo/test/testx.go b/misc/cgo/test/testx.go index 42979b5f4d..98c48fa383 100644 --- a/misc/cgo/test/testx.go +++ b/misc/cgo/test/testx.go @@ -159,7 +159,7 @@ func Add(x int) { } func testCthread(t *testing.T) { - if runtime.GOOS == "darwin" && (runtime.GOARCH == "arm" || runtime.GOARCH == "arm64") { + if runtime.GOOS == "darwin" && runtime.GOARCH == "arm64" { t.Skip("the iOS exec wrapper is unable to properly handle the panic from Add") } sum.i = 0 diff --git a/misc/cgo/testcarchive/carchive_test.go b/misc/cgo/testcarchive/carchive_test.go index 82a1a5a54c..b4a046c4bc 100644 --- a/misc/cgo/testcarchive/carchive_test.go +++ b/misc/cgo/testcarchive/carchive_test.go @@ -134,7 +134,7 @@ func testMain(m *testing.M) int { } else { switch GOOS { case "darwin": - if GOARCH == "arm" || GOARCH == "arm64" { + if GOARCH == "arm64" { libbase += "_shared" } case "dragonfly", "freebsd", "linux", "netbsd", "openbsd", "solaris", "illumos": @@ -305,7 +305,7 @@ func TestEarlySignalHandler(t *testing.T) { switch GOOS { case "darwin": switch GOARCH { - case "arm", "arm64": + case "arm64": t.Skipf("skipping on %s/%s; see https://golang.org/issue/13701", GOOS, GOARCH) } case "windows": @@ -487,7 +487,7 @@ func checkSignalForwardingTest(t *testing.T) { switch GOOS { case "darwin": switch GOARCH { - case "arm", "arm64": + case "arm64": t.Skipf("skipping on %s/%s; see https://golang.org/issue/13701", GOOS, GOARCH) } case "windows": @@ -603,7 +603,7 @@ func TestExtar(t *testing.T) { if runtime.Compiler == "gccgo" { t.Skip("skipping -extar test when using gccgo") } - if runtime.GOOS == "darwin" && (runtime.GOARCH == "arm" || runtime.GOARCH == "arm64") { + if runtime.GOOS == "darwin" && runtime.GOARCH == "arm64" { t.Skip("shell scripts are not executable on iOS hosts") } diff --git a/misc/cgo/testcshared/cshared_test.go b/misc/cgo/testcshared/cshared_test.go index cb95153abf..bd4d341820 100644 --- a/misc/cgo/testcshared/cshared_test.go +++ b/misc/cgo/testcshared/cshared_test.go @@ -108,7 +108,7 @@ func testMain(m *testing.M) int { libgodir := GOOS + "_" + GOARCH switch GOOS { case "darwin": - if GOARCH == "arm" || GOARCH == "arm64" { + if GOARCH == "arm64" { libgodir += "_shared" } case "dragonfly", "freebsd", "linux", "netbsd", "openbsd", "solaris", "illumos": diff --git a/misc/cgo/testso/so_test.go b/misc/cgo/testso/so_test.go index f94017025c..bdd6bd8468 100644 --- a/misc/cgo/testso/so_test.go +++ b/misc/cgo/testso/so_test.go @@ -20,7 +20,7 @@ import ( func requireTestSOSupported(t *testing.T) { t.Helper() switch runtime.GOARCH { - case "arm", "arm64": + case "arm64": if runtime.GOOS == "darwin" { t.Skip("No exec facility on iOS.") } diff --git a/misc/cgo/testsovar/so_test.go b/misc/cgo/testsovar/so_test.go index f94017025c..bdd6bd8468 100644 --- a/misc/cgo/testsovar/so_test.go +++ b/misc/cgo/testsovar/so_test.go @@ -20,7 +20,7 @@ import ( func requireTestSOSupported(t *testing.T) { t.Helper() switch runtime.GOARCH { - case "arm", "arm64": + case "arm64": if runtime.GOOS == "darwin" { t.Skip("No exec facility on iOS.") } diff --git a/misc/ios/clangwrap.sh b/misc/ios/clangwrap.sh index c69a9bf355..5fdbb6db4a 100755 --- a/misc/ios/clangwrap.sh +++ b/misc/ios/clangwrap.sh @@ -8,9 +8,7 @@ export IPHONEOS_DEPLOYMENT_TARGET=5.1 # cmd/cgo doesn't support llvm-gcc-4.2, so we have to use clang. CLANG=`xcrun --sdk $SDK --find clang` -if [ "$GOARCH" == "arm" ]; then - CLANGARCH="armv7" -elif [ "$GOARCH" == "arm64" ]; then +if [ "$GOARCH" == "arm64" ]; then CLANGARCH="arm64" else echo "unknown GOARCH=$GOARCH" >&2 |