diff options
| author | Cherry Mui <cherryyz@google.com> | 2024-11-01 12:04:49 -0400 |
|---|---|---|
| committer | Cherry Mui <cherryyz@google.com> | 2024-11-13 20:09:36 +0000 |
| commit | 4ffa2aecc119bd70f4765cbb9b911f47dbadd117 (patch) | |
| tree | 07d4063fb3ad87dd2e583e96664448c4a672e177 /lib | |
| parent | 26e1010372216d013a0279be6b453fd680130881 (diff) | |
| download | go-4ffa2aecc119bd70f4765cbb9b911f47dbadd117.tar.xz | |
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>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/wasm/wasm_exec.js | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/wasm/wasm_exec.js b/lib/wasm/wasm_exec.js index af7e28f5f4..ec96d42db5 100644 --- a/lib/wasm/wasm_exec.js +++ b/lib/wasm/wasm_exec.js @@ -217,6 +217,7 @@ } const testCallExport = (a, b) => { + this._inst.exports.testExport0(); return this._inst.exports.testExport(a, b); } |