]> Cypherpunks repositories - gostls13.git/commitdiff
test: re-enable atomic intrinsic related tests on riscv64
authorJoel Sing <joel@sing.id.au>
Sun, 15 Mar 2020 15:58:23 +0000 (02:58 +1100)
committerJoel Sing <joel@sing.id.au>
Wed, 25 Mar 2020 01:11:15 +0000 (01:11 +0000)
riscv64 now has atomic intrinsics, so re-enable the atomic intrinsic tests.

Fixes #36765

Change-Id: I838f27570a94d7fa5774c43f1ca5f4df2ca104cf
Reviewed-on: https://go-review.googlesource.com/c/go/+/223560
Reviewed-by: Keith Randall <khr@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
test/inline_sync.go
test/intrinsic_atomic.go

index c37cb136f7b0fcb4bcd046b8b30d9702dc8afe04..30b436af41289cb4b016bc34b401dd9e1a9880a2 100644 (file)
@@ -1,4 +1,4 @@
-// +build !nacl,!386,!wasm,!arm,!riscv64,!gcflags_noopt
+// +build !nacl,!386,!wasm,!arm,!gcflags_noopt
 // errorcheck -0 -m
 
 // Copyright 2019 The Go Authors. All rights reserved.
@@ -14,9 +14,6 @@
 // of the sync fast paths. This test should be re-enabled once the problem
 // is solved.
 
-// TODO(jsing): Re-enable on riscv64 when it has atomic intrinsics - see
-// golang.org/issue/36765
-
 package foo
 
 import (
index a90056e1f15fff1c485bfb027f94f3516d114d1a..61911b7a46e2ed2036d48be83fc0ad82a76b9a92 100644 (file)
@@ -1,5 +1,5 @@
 // errorcheck -0 -d=ssa/intrinsics/debug
-// +build amd64 arm64 mips mipsle mips64 mips64le ppc64 ppc64le s390x
+// +build amd64 arm64 mips mipsle mips64 mips64le ppc64 ppc64le riscv64 s390x
 
 // Copyright 2016 The Go Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style