saulecabrera opened PR #6204 from saulecabrera:winch-x64-improve-abi-compatibility
to bytecodealliance:main
:
Follow up to https://github.com/bytecodealliance/wasmtime/pull/6119
This commit improves ABI support in Winch's trampolines mainly by:
- Adding support for the
fastcall
calling convention.- By storing/restoring callee-saved registers.
One of the explicit goals of this change is to make tests available in the x86_64 target as a whole and remove the need exclude the windows target.
This commit also introduces a
CallingConvention
enum, to better reflect the subset of calling conventions that are supported by Winch.
prtest:full
<!--
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 ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
saulecabrera updated PR #6204.
saulecabrera requested cfallin for a review on PR #6204.
saulecabrera has marked PR #6204 as ready for review.
saulecabrera requested wasmtime-compiler-reviewers for a review on PR #6204.
saulecabrera requested alexcrichton for a review on PR #6204.
saulecabrera requested wasmtime-core-reviewers for a review on PR #6204.
saulecabrera updated PR #6204.
saulecabrera edited PR #6204:
Follow up to https://github.com/bytecodealliance/wasmtime/pull/6119
This commit improves ABI support in Winch's trampolines mainly by:
- Adding support for the
fastcall
calling convention.- By storing/restoring callee-saved registers.
One of the explicit goals of this change is to make tests available in the x86_64 target as a whole and remove the need exclude the windows target.
This commit also introduces a
CallingConvention
enum, to better reflect the subset of calling conventions that are supported by Winch.<!--
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 ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
alexcrichton submitted PR review.
cfallin submitted PR review.
cfallin submitted PR review.
cfallin created PR review comment:
s/calle/callee/
cfallin created PR review comment:
s/support/supporting/
saulecabrera updated PR #6204.
saulecabrera submitted PR review.
saulecabrera created PR review comment:
Fixed, thanks!
saulecabrera submitted PR review.
saulecabrera created PR review comment:
Fixed, thanks!
saulecabrera merged PR #6204.
Last updated: Nov 22 2024 at 16:03 UTC