From c6d142c4a7c0fa9fc11ad511de8f7d7412b662f7 Mon Sep 17 00:00:00 2001 From: "Paul E. Murphy" Date: Mon, 20 May 2024 14:44:21 -0500 Subject: cmd/compile/internal/ssa: fix ppc64 merging of (CLRLSLDI (SRD ...)) The rotate value was not correctly converted from a 64 bit to 32 bit rotate. This caused a miscompile of golang.org/x/text/unicode/runenames.Names. Fixes #67526 Change-Id: Ief56fbab27ccc71cd4c01117909bfee7f60a2ea1 Reviewed-on: https://go-review.googlesource.com/c/go/+/586915 Reviewed-by: Dmitri Shuralyov LUCI-TryBot-Result: Go LUCI Reviewed-by: Lynn Boger Reviewed-by: Carlos Amedee --- test/codegen/shift.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'test/codegen') diff --git a/test/codegen/shift.go b/test/codegen/shift.go index dd91a1db98..6a2a6c40cd 100644 --- a/test/codegen/shift.go +++ b/test/codegen/shift.go @@ -453,7 +453,7 @@ func checkMergedShifts32(a [256]uint32, b [256]uint64, u uint32, v uint32) { b[2] = b[v>>25] } -func checkMergedShifts64(a [256]uint32, b [256]uint64, v uint64) { +func checkMergedShifts64(a [256]uint32, b [256]uint64, c [256]byte, v uint64) { // ppc64x: -"CLRLSLDI", "RLWNM\t[$]10, R[0-9]+, [$]22, [$]29, R[0-9]+" a[0] = a[uint8(v>>24)] // ppc64x: "SRD", "CLRLSLDI", -"RLWNM" @@ -474,6 +474,10 @@ func checkMergedShifts64(a [256]uint32, b [256]uint64, v uint64) { b[1] = b[(v>>20)&0xFF] // ppc64x: "RLWNM", -"SLD" b[2] = b[((uint64((uint32(v) >> 21)) & 0x3f) << 4)] + // ppc64x: "RLWNM\t[$]11, R[0-9]+, [$]10, [$]15" + c[0] = c[((v>>5)&0x3F)<<16] + // ppc64x: "RLWNM\t[$]0, R[0-9]+, [$]19, [$]24" + c[1] = c[((v>>7)&0x3F)<<7] } func checkShiftMask(a uint32, b uint64, z []uint32, y []uint64) { -- cgit v1.3-5-g9baa