]> Cypherpunks repositories - gostls13.git/commitdiff
[dev.regabi] test: enable finalizer tests on !amd64
authorMatthew Dempsky <mdempsky@google.com>
Tue, 5 Jan 2021 16:37:41 +0000 (08:37 -0800)
committerMatthew Dempsky <mdempsky@google.com>
Tue, 5 Jan 2021 21:11:31 +0000 (21:11 +0000)
The gc implementation has had precise GC for a while now, so we can
enable these tests more broadly.

Confirmed that they still fail with gccgo 10.2.1.

Change-Id: Ic1c0394ab832024a99e34163c422941a3706e1a2
Reviewed-on: https://go-review.googlesource.com/c/go/+/281542
Trust: Matthew Dempsky <mdempsky@google.com>
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Cuong Manh Le <cuong.manhle.vn@gmail.com>
test/deferfin.go
test/fixedbugs/issue5493.go

index 80372916d20b303303483f6ea36911b20482c8dc..1312bbbe71b04ac195330dc9151c84cfcf87031e 100644 (file)
@@ -18,12 +18,8 @@ import (
 var sink func()
 
 func main() {
-       // Does not work on 32-bits due to partially conservative GC.
+       // Does not work with gccgo, due to partially conservative GC.
        // Try to enable when we have fully precise GC.
-       if runtime.GOARCH != "amd64" {
-               return
-       }
-       // Likewise for gccgo.
        if runtime.Compiler == "gccgo" {
                return
        }
@@ -60,4 +56,3 @@ func main() {
                panic("not all finalizers are called")
        }
 }
-
index 2ee0398af2c894f4939f594a758eadb9a82d71d5..8f771bc2db369c9baa9d80cd773c1d62d1cc0225 100644 (file)
@@ -14,6 +14,7 @@ import (
 )
 
 const N = 10
+
 var count int64
 
 func run() error {
@@ -31,10 +32,9 @@ func run() error {
 }
 
 func main() {
-       // Does not work on 32-bits, or with gccgo, due to partially
-       // conservative GC.
+       // Does not work with gccgo, due to partially conservative GC.
        // Try to enable when we have fully precise GC.
-       if runtime.GOARCH != "amd64" || runtime.Compiler == "gccgo" {
+       if runtime.Compiler == "gccgo" {
                return
        }
        count = N
@@ -56,4 +56,3 @@ func main() {
                panic("not all finalizers are called")
        }
 }
-