]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: remove arbitrary GOARCH constraints in finalizer tests
authorBryan C. Mills <bcmills@google.com>
Thu, 8 Dec 2022 13:44:51 +0000 (08:44 -0500)
committerGopher Robot <gobot@golang.org>
Thu, 19 Jan 2023 20:45:58 +0000 (20:45 +0000)
These tests were only run on GOARCH=amd64, but the rationale given in
CL 11858043 was GC precision on 32-bit platforms. Today, we have far
more 64-bit platforms than just amd64, and I believe that GC precision
on 32-bit platforms has been substantially improved as well.
The GOARCH restriction seems unnecessary.

Updates #57166.
Updates #5368.

Change-Id: I45c608b6fa721012792c96d4ed94a6d772b90210
Reviewed-on: https://go-review.googlesource.com/c/go/+/456120
Run-TryBot: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Bryan Mills <bcmills@google.com>
Reviewed-by: Austin Clements <austin@google.com>
src/runtime/mfinal_test.go

index 61d625ac27775516abfc1a27134fdeb4d395fbb7..87d31c472c1b01110348d990009b3b68f5584b3a 100644 (file)
@@ -21,10 +21,6 @@ type Tinter interface {
 }
 
 func TestFinalizerType(t *testing.T) {
-       if runtime.GOARCH != "amd64" {
-               t.Skipf("Skipping on non-amd64 machine")
-       }
-
        ch := make(chan bool, 10)
        finalize := func(x *int) {
                if *x != 97531 {
@@ -82,9 +78,6 @@ type bigValue struct {
 }
 
 func TestFinalizerInterfaceBig(t *testing.T) {
-       if runtime.GOARCH != "amd64" {
-               t.Skipf("Skipping on non-amd64 machine")
-       }
        ch := make(chan bool)
        done := make(chan bool, 1)
        go func() {