]> Cypherpunks repositories - gostls13.git/commit
cmd/internal/obj/wasm: correct return PC for frameless wasmexport wrappers
authorCherry Mui <cherryyz@google.com>
Fri, 1 Nov 2024 16:04:49 +0000 (12:04 -0400)
committerCherry Mui <cherryyz@google.com>
Wed, 13 Nov 2024 20:09:36 +0000 (20:09 +0000)
commit4ffa2aecc119bd70f4765cbb9b911f47dbadd117
tree07d4063fb3ad87dd2e583e96664448c4a672e177
parent26e1010372216d013a0279be6b453fd680130881
cmd/internal/obj/wasm: correct return PC for frameless wasmexport wrappers

For a wasmexport wrapper, we generate a call to the actual
exported Go function, and use the wrapper function's PC 1 as the
(fake) return address. This address is not used for returning,
which is handled by the Wasm call stack. It is used for stack
unwinding, and PC 1 makes it past the prologue and therefore has
the right SP delta. But if the function has no arguments and
results, the wrapper is frameless, with no prologue, and PC 1
doesn't exist. This causes the unwinder to fail. In this case, we
put PC 0, which also has the correct SP delta (0).

Fixes #69584.

Change-Id: Ic047a6e62100db540b5099cc5a56a1d0f16d58b9
Reviewed-on: https://go-review.googlesource.com/c/go/+/624000
Reviewed-by: David Chase <drchase@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
lib/wasm/wasm_exec.js
src/cmd/internal/obj/wasm/wasmobj.go
src/syscall/js/js_test.go