]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: unbreak linux/riscv64 following regabi merge
authorJoel Sing <joel@sing.id.au>
Wed, 17 Feb 2021 09:34:34 +0000 (20:34 +1100)
committerJoel Sing <joel@sing.id.au>
Thu, 18 Feb 2021 04:30:23 +0000 (04:30 +0000)
Unbreak the linux/riscv64 port by storing the zero value register to memory,
rather than the current code that is moving a zero intermediate to the stack
pointer register (ideally this should be caught by the assembler). This was
broken in CL#272568.

On riscv64 a zero immediate value cannot be moved directly to memory, rather
a register needs to be loaded with zero and then stored. Alternatively, the
the zero value register (aka X0) can be used directly.

Change-Id: Id57121541d50c9993cec5c2270b638b184ab9bc1
Reviewed-on: https://go-review.googlesource.com/c/go/+/292894
Trust: Joel Sing <joel@sing.id.au>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Run-TryBot: Michael Knyszek <mknyszek@google.com>
TryBot-Result: Go Bot <gobot@golang.org>

src/runtime/asm_riscv64.s

index 31e324d6775926d000e705ac01a0d9f788047d75..3d0349471a2797304aef48e63d9c9225328264b8 100644 (file)
@@ -449,7 +449,7 @@ TEXT callRet<>(SB), NOSPLIT, $40-0
        MOV     A1, 16(X2)
        MOV     A3, 24(X2)
        MOV     A2, 32(X2)
-       MOV     $0, 40(X2)
+       MOV     ZERO, 40(X2)
        CALL    runtime·reflectcallmove(SB)
        RET