From 7f3569c33acb2e1a23c55a29ddff2203e39c0176 Mon Sep 17 00:00:00 2001 From: Matthew Dempsky Date: Thu, 11 Aug 2022 15:05:59 -0700 Subject: [PATCH] cmd/compile/internal/ssa: include "ppc64" in has-regabi arches list While here, rename helper function to follow Go naming idioms: https://github.com/golang/go/wiki/CodeReviewComments#initialisms Fixes #53456. Change-Id: I041dd25968e192ea2b430d58e3348626a970f92b Reviewed-on: https://go-review.googlesource.com/c/go/+/422957 Reviewed-by: Cuong Manh Le Reviewed-by: Lynn Boger TryBot-Result: Gopher Robot Reviewed-by: Keith Randall Run-TryBot: Matthew Dempsky --- src/cmd/compile/internal/ssa/debug_lines_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/cmd/compile/internal/ssa/debug_lines_test.go b/src/cmd/compile/internal/ssa/debug_lines_test.go index 2451e2487b0..47772cf1a51 100644 --- a/src/cmd/compile/internal/ssa/debug_lines_test.go +++ b/src/cmd/compile/internal/ssa/debug_lines_test.go @@ -45,9 +45,9 @@ func testGoArch() string { return *testGoArchFlag } -func hasRegisterAbi() bool { +func hasRegisterABI() bool { switch testGoArch() { - case "amd64", "arm64", "ppc64le", "riscv": + case "amd64", "arm64", "ppc64", "ppc64le", "riscv": return true } return false @@ -62,7 +62,7 @@ func unixOnly(t *testing.T) { // testDebugLinesDefault removes the first wanted statement on architectures that are not (yet) register ABI. func testDebugLinesDefault(t *testing.T, gcflags, file, function string, wantStmts []int, ignoreRepeats bool) { unixOnly(t) - if !hasRegisterAbi() { + if !hasRegisterABI() { wantStmts = wantStmts[1:] } testDebugLines(t, gcflags, file, function, wantStmts, ignoreRepeats)