Skip to content

Commit d4dbad5

Browse files
Bryan Millsgopherbot
Bryan Mills
authored andcommitted
Revert "cmd/compile/internal: fix test error on loong64"
This reverts CL 367043. Reason for revert: auto-submitted prematurely, breaking tests on most builders. Change-Id: I6da319fb042b629bcd6f549be638497a357e7d28 Reviewed-on: https://go-review.googlesource.com/c/go/+/399795 Run-TryBot: Bryan Mills <[email protected]> Auto-Submit: Bryan Mills <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]> TryBot-Result: Gopher Robot <[email protected]>
1 parent ace7672 commit d4dbad5

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/cmd/compile/internal/logopt/logopt_test.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ func s15a8(x *[15]int64) [15]int64 {
155155
arches := []string{runtime.GOARCH}
156156
goos0 := runtime.GOOS
157157
if runtime.GOARCH == "amd64" { // Test many things with "linux" (wasm will get "js")
158-
arches = []string{"arm", "arm64", "386", "amd64", "mips", "mips64", "loong64", "ppc64le", "riscv64", "s390x", "wasm"}
158+
arches = []string{"arm", "arm64", "386", "amd64", "mips", "mips64", "ppc64le", "riscv64", "s390x", "wasm"}
159159
goos0 = "linux"
160160
}
161161

src/cmd/compile/internal/test/inl_test.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -163,10 +163,10 @@ func TestIntendedInlining(t *testing.T) {
163163
},
164164
}
165165

166-
if runtime.GOARCH != "386" && runtime.GOARCH != "loong64" && runtime.GOARCH != "mips64" && runtime.GOARCH != "mips64le" && runtime.GOARCH != "riscv64" {
166+
if runtime.GOARCH != "386" && runtime.GOARCH != "mips64" && runtime.GOARCH != "mips64le" && runtime.GOARCH != "riscv64" {
167167
// nextFreeFast calls sys.Ctz64, which on 386 is implemented in asm and is not inlinable.
168168
// We currently don't have midstack inlining so nextFreeFast is also not inlinable on 386.
169-
// On loong64, mips64x and riscv64, Ctz64 is not intrinsified and causes nextFreeFast too expensive
169+
// On mips64x and riscv64, Ctz64 is not intrinsified and causes nextFreeFast too expensive
170170
// to inline (Issue 22239).
171171
want["runtime"] = append(want["runtime"], "nextFreeFast")
172172
}

0 commit comments

Comments
 (0)