cfallin opened PR #14642 from cfallin:fix-tail-unwinding to bytecodealliance:main:
While Wasmtime doesn't use the DWARF unwind info / frame definitions, it is still useful to try to get them to be as correct as possible, because native debugging and especially native profiling (
perf record,samply, etc) use this info.Previously, the definition of the CFA did not quite stay in sync with a valid frame when we move the frame due to a resizing argument area on tail-calls. This PR reworks the metadata to instead define the CFA in terms of the FP and leave FP unchanged until we switch to the new frame.
<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease review the Bytecode Alliance's AI tool usage policy at
https://github.com/bytecodealliance/governance/blob/main/AI_TOOL_POLICY.mdPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
cfallin requested wasmtime-compiler-reviewers for a review on PR #14642.
cfallin requested alexcrichton for a review on PR #14642.
:thumbs_up: alexcrichton submitted PR review.
alexcrichton has enabled auto merge for PR #14642.
alexcrichton added PR #14642 Cranelift: fix unwind info around tail calls. to the merge queue
:check: alexcrichton merged PR #14642.
alexcrichton removed PR #14642 Cranelift: fix unwind info around tail calls. from the merge queue
Last updated: Oct 11 2026 at 04:10 UTC