From: Joel Sing <joel@sing.id.au>
Date: Wed, 12 Mar 2025 12:11:30 +0000 (+1100)
Subject: test/codegen: remove plan9/amd64 specific array zeroing/copying tests
X-Git-Tag: go1.25rc1~724
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=af92bb594d6daaacdd2d03549911fbec8ae65900;p=gostls13.git

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 <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>
---

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
 }