Commit ba333a30 authored by Elias Naur's avatar Elias Naur

cmd/go: remove limits on parallel tasks in the Go command for iOS

Ther darwin/arm{,64} exec wrapper now limits the number of concurrent
executions to 1, so remove the higher level parallel task limit from
the Go command.

Change-Id: Id84f65c3908305bde0452b3c8db6df8c5a8881bb
Reviewed-on: https://go-review.googlesource.com/21100Reviewed-by: default avatarDavid Crawshaw <crawshaw@golang.org>
parent 95add73a
// Copyright 2011 The Go Authors. All rights reserved. // Copyright 2011 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style // Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file. // license that can be found in the LICENSE file.
...@@ -86,8 +86,7 @@ and test commands: ...@@ -86,8 +86,7 @@ and test commands:
-p n -p n
the number of programs, such as build commands or the number of programs, such as build commands or
test binaries, that can be run in parallel. test binaries, that can be run in parallel.
The default is the number of CPUs available, except The default is the number of CPUs available.
on darwin/arm which defaults to 1.
-race -race
enable data race detection. enable data race detection.
Supported only on linux/amd64, freebsd/amd64, darwin/amd64 and windows/amd64. Supported only on linux/amd64, freebsd/amd64, darwin/amd64 and windows/amd64.
...@@ -577,6 +576,7 @@ syntax of package template. The default output is equivalent to -f ...@@ -577,6 +576,7 @@ syntax of package template. The default output is equivalent to -f
CXXFiles []string // .cc, .cxx and .cpp source files CXXFiles []string // .cc, .cxx and .cpp source files
MFiles []string // .m source files MFiles []string // .m source files
HFiles []string // .h, .hh, .hpp and .hxx source files HFiles []string // .h, .hh, .hpp and .hxx source files
FFiles []string // .f, .F, .for and .f90 Fortran source files
SFiles []string // .s source files SFiles []string // .s source files
SwigFiles []string // .swig files SwigFiles []string // .swig files
SwigCXXFiles []string // .swigcxx files SwigCXXFiles []string // .swigcxx files
...@@ -586,6 +586,7 @@ syntax of package template. The default output is equivalent to -f ...@@ -586,6 +586,7 @@ syntax of package template. The default output is equivalent to -f
CgoCFLAGS []string // cgo: flags for C compiler CgoCFLAGS []string // cgo: flags for C compiler
CgoCPPFLAGS []string // cgo: flags for C preprocessor CgoCPPFLAGS []string // cgo: flags for C preprocessor
CgoCXXFLAGS []string // cgo: flags for C++ compiler CgoCXXFLAGS []string // cgo: flags for C++ compiler
CgoFFLAGS []string // cgo: flags for Fortran compiler
CgoLDFLAGS []string // cgo: flags for linker CgoLDFLAGS []string // cgo: flags for linker
CgoPkgConfig []string // cgo: pkg-config names CgoPkgConfig []string // cgo: pkg-config names
...@@ -1563,6 +1564,7 @@ Here is another example where the ordering of the output is ignored: ...@@ -1563,6 +1564,7 @@ Here is another example where the ordering of the output is ignored:
for _, value := range Perm(4) { for _, value := range Perm(4) {
fmt.Println(value) fmt.Println(value)
} }
// Unordered output: 4 // Unordered output: 4
// 2 // 2
// 1 // 1
......
...@@ -65,8 +65,7 @@ and test commands: ...@@ -65,8 +65,7 @@ and test commands:
-p n -p n
the number of programs, such as build commands or the number of programs, such as build commands or
test binaries, that can be run in parallel. test binaries, that can be run in parallel.
The default is the number of CPUs available, except The default is the number of CPUs available.
on darwin/arm which defaults to 1.
-race -race
enable data race detection. enable data race detection.
Supported only on linux/amd64, freebsd/amd64, darwin/amd64 and windows/amd64. Supported only on linux/amd64, freebsd/amd64, darwin/amd64 and windows/amd64.
...@@ -145,17 +144,6 @@ func init() { ...@@ -145,17 +144,6 @@ func init() {
addBuildFlags(cmdBuild) addBuildFlags(cmdBuild)
addBuildFlags(cmdInstall) addBuildFlags(cmdInstall)
if buildContext.GOOS == "darwin" {
switch buildContext.GOARCH {
case "arm", "arm64":
// darwin/arm cannot run multiple tests simultaneously.
// Parallelism is limited in go_darwin_arm_exec, but
// also needs to be limited here so go test std does not
// timeout tests that waiting to run.
buildP = 1
}
}
} }
// Flags set by multiple commands. // Flags set by multiple commands.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment