aboutsummaryrefslogtreecommitdiff
path: root/src/runtime/panic.go
diff options
context:
space:
mode:
authorThan McIntosh <thanm@google.com>2021-04-05 16:50:38 -0400
committerThan McIntosh <thanm@google.com>2021-04-05 23:21:16 +0000
commitb2389ad3ce7254784db5f4005805714e87ffab85 (patch)
tree5c4ce785e0b23ec6e3aa8ecb10c1a63b295ea783 /src/runtime/panic.go
parentd446cb7cff024412c0a80c3971dac3049db9f18c (diff)
downloadgo-b2389ad3ce7254784db5f4005805714e87ffab85.tar.xz
cmd/compile: fix for zerorange on plan9-amd64
In CL 305829 a problematic change was made to the compiler's amd64-specific "zerorange" function. In zerorange the compiler uses different sets of strategies depending on the size of the stack frame it needs to zero; turns out that only on plan9-amd64 was it hitting the final fallback strategy, which is a REPSTOSQ instruction. REPSTOSQ takes RAX as an input, hence the changes made in CL 305829 (switching to R13) were incorrect. This patch restores the zerorange REPSTOSQ sequence (back to use RAX). This is going to be an interim solution, since long term we need to avoid touching RAX in the function prolog (since if the new register ABI is in effect, it will hold a live value). Fixes #45372. Change-Id: Ic89a6a2a76d6e03b9fbda99275101e96b70fdf5d Reviewed-on: https://go-review.googlesource.com/c/go/+/307469 Trust: Than McIntosh <thanm@google.com> Run-TryBot: Than McIntosh <thanm@google.com> TryBot-Result: Go Bot <gobot@golang.org> Reviewed-by: David du Colombier <0intro@gmail.com> Reviewed-by: Cherry Zhang <cherryyz@google.com>
Diffstat (limited to 'src/runtime/panic.go')
0 files changed, 0 insertions, 0 deletions