From af92bb594d6daaacdd2d03549911fbec8ae65900 Mon Sep 17 00:00:00 2001 From: Joel Sing Date: Wed, 12 Mar 2025 23:11:30 +1100 Subject: [PATCH] test/codegen: remove plan9/amd64 specific array zeroing/copying tests 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 Reviewed-by: David Chase Reviewed-by: Keith Randall Reviewed-by: Keith Randall Reviewed-by: Brad Fitzpatrick Auto-Submit: Keith Randall --- test/codegen/floats.go | 6 ------ 1 file changed, 6 deletions(-) diff --git a/test/codegen/floats.go b/test/codegen/floats.go index 2a5cf39957..d04202d394 100644 --- a/test/codegen/floats.go +++ b/test/codegen/floats.go @@ -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 } -- 2.51.0