diff options
| author | Austin Clements <austin@google.com> | 2023-05-10 15:51:41 -0400 |
|---|---|---|
| committer | Austin Clements <austin@google.com> | 2023-05-12 12:35:07 +0000 |
| commit | b679e31cdbf09658388795c7b15c64fce86d6bd2 (patch) | |
| tree | 3e8c08a5be06c4fd136e888c5490d0670a2bd239 /src/cmd/dist/test.go | |
| parent | 3943fc14d35c117f2256fb789743d9f075c9aba5 (diff) | |
| download | go-b679e31cdbf09658388795c7b15c64fce86d6bd2.tar.gz go-b679e31cdbf09658388795c7b15c64fce86d6bd2.zip | |
test/bench: delete
Russ added test/bench/go1 in CL 5484071 to have a stable suite of
programs to use as benchmarks. For the compiler and runtime we had
back then, those were reasonable benchmarks, but the compiler and
runtime are now far more sophisticated and these benchmarks no longer
have good coverage. We also now have better benchmark suites
maintained outside the repo (e.g., golang.org/x/benchmarks). Keeping
test/bench/go1 at this point is actively misleading.
Indirectly related to #37486, as this also removes the last package
dist test runs outside of src/.
Change-Id: I2867ef303fe48a02acce58ace4ee682add8acdbf
Reviewed-on: https://go-review.googlesource.com/c/go/+/494193
Run-TryBot: Austin Clements <austin@google.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
Reviewed-by: Russ Cox <rsc@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Diffstat (limited to 'src/cmd/dist/test.go')
| -rw-r--r-- | src/cmd/dist/test.go | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/cmd/dist/test.go b/src/cmd/dist/test.go index 7ec10aecac..9cb640737a 100644 --- a/src/cmd/dist/test.go +++ b/src/cmd/dist/test.go @@ -852,11 +852,6 @@ func (t *tester) registerTests() { } if goos != "android" && !t.iOS() { - // There are no tests in this directory, only benchmarks. - // Check that the test binary builds. - t.registerTest("bench_go1", "", &goTest{dir: "../test/bench/go1"}) - } - if goos != "android" && !t.iOS() { // Only start multiple test dir shards on builders, // where they get distributed to multiple machines. // See issues 20141 and 31834. |