]> Cypherpunks repositories - gostls13.git/commitdiff
test/codegen: remove plan9/amd64 specific array zeroing/copying tests
authorJoel Sing <joel@sing.id.au>
Wed, 12 Mar 2025 12:11:30 +0000 (23:11 +1100)
committerGopher Robot <gobot@golang.org>
Thu, 13 Mar 2025 12:19:13 +0000 (05:19 -0700)
The compiler previously avoided the use of MOVUPS on plan9/amd64. This
was changed in CL 655875, however the codegen tests were not updated
and now fail (seemingly the full codegen tests do not run anywhere,
not even on the longtest builders).

Change-Id: I388b60e7b0911048d4949c5029347f9801c018a9
Reviewed-on: https://go-review.googlesource.com/c/go/+/656997
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: David Chase <drchase@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
Reviewed-by: Keith Randall <khr@google.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Auto-Submit: Keith Randall <khr@google.com>

test/codegen/floats.go

index 2a5cf3995781e61d885fdeea8f8e007eb5d1c91c..d04202d39423d14115d1fb071b1081d6bd36bc2e 100644 (file)
@@ -149,20 +149,14 @@ func CmpWithAdd(a float64, b float64) bool {
 //    Non-floats    //
 // ---------------- //
 
-// We should make sure that the compiler doesn't generate floating point
-// instructions for non-float operations on Plan 9, because floating point
-// operations are not allowed in the note handler.
-
 func ArrayZero() [16]byte {
        // amd64:"MOVUPS"
-       // plan9/amd64/:-"MOVUPS"
        var a [16]byte
        return a
 }
 
 func ArrayCopy(a [16]byte) (b [16]byte) {
        // amd64:"MOVUPS"
-       // plan9/amd64/:-"MOVUPS"
        b = a
        return
 }