cmd/coordinator: make linux-amd64-racecompile rebuild std/cmd with -c=128

Updates golang/go#19962

Change-Id: I8b938a39b33d4e6b9b8dc5bc0802da95e889ee46
Reviewed-on: https://go-review.googlesource.com/41613
Reviewed-by: Josh Bleecher Snyder <josharian@gmail.com>
This commit is contained in:
Brad Fitzpatrick 2017-04-24 21:14:05 +00:00
Родитель 97a4f81784
Коммит 4f5eef0322
2 изменённых файлов: 35 добавлений и 0 удалений

Просмотреть файл

@ -1443,6 +1443,9 @@ func (st *buildStatus) onceInitHelpersFunc() {
// make.bash if it exists (anything can SplitMakeRun) and that the // make.bash if it exists (anything can SplitMakeRun) and that the
// snapshot exists. // snapshot exists.
func (st *buildStatus) useSnapshot() bool { func (st *buildStatus) useSnapshot() bool {
if st.conf.SkipSnapshot {
return false
}
st.mu.Lock() st.mu.Lock()
defer st.mu.Unlock() defer st.mu.Unlock()
if st.useSnapshotMemo != nil { if st.useSnapshotMemo != nil {
@ -1816,6 +1819,36 @@ func (st *buildStatus) runMake(bc *buildlet.Client, goroot string, w io.Writer)
sp.done(nil) sp.done(nil)
} }
if st.name == "linux-amd64-racecompile" {
return st.runConcurrentGoBuildStdCmd(bc)
}
return nil, nil
}
// runConcurrentGoBuildStdCmd is a step specific only to the
// "linux-amd64-racecompile" builder to exercise the Go 1.9's new
// concurrent compilation. It re-builds the standard library and tools
// with -gcflags=-c=128 using a race-enabled cmd/compile (built by
// caller, runMake, per builder config).
// The idea is that this might find data races in cmd/compile.
func (st *buildStatus) runConcurrentGoBuildStdCmd(bc *buildlet.Client) (remoteErr, err error) {
span := st.createSpan("go_build_c128_std_cmd")
remoteErr, err = bc.Exec("go/bin/go", buildlet.ExecOpts{
Output: st,
ExtraEnv: append(st.conf.Env(), "GOBIN="),
Debug: true,
Args: []string{"build", "-a", "-gcflags=-c=128", "std", "cmd"},
})
if err != nil {
span.done(err)
return nil, err
}
if remoteErr != nil {
span.done(remoteErr)
return fmt.Errorf("go build failed: %v", remoteErr), nil
}
span.done(nil)
return nil, nil return nil, nil
} }

Просмотреть файл

@ -856,6 +856,8 @@ func init() {
Name: "linux-amd64-racecompile", Name: "linux-amd64-racecompile",
HostType: "host-linux-kubestd", HostType: "host-linux-kubestd",
CompileOnly: true, CompileOnly: true,
SkipSnapshot: true,
StopAfterMake: true,
InstallRacePackages: []string{"cmd/compile"}, InstallRacePackages: []string{"cmd/compile"},
Notes: "race-enabled cmd/compile", Notes: "race-enabled cmd/compile",
}) })