fitzgen opened PR #6749 from fitzgen:riscv64-tail-calls
to bytecodealliance:main
:
Builds on top of https://github.com/bytecodealliance/wasmtime/pull/6723
<!--
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
-->
fitzgen requested elliottt for a review on PR #6749.
fitzgen requested wasmtime-compiler-reviewers for a review on PR #6749.
elliottt submitted PR review.
elliottt created PR review comment:
Should riscv64 also have tail calls enabled when fuzzing?
fitzgen submitted PR review.
fitzgen created PR review comment:
Yes it should. Good catch, thanks!
afonso360 submitted PR review.
afonso360 created PR review comment:
I've been running this on the fuzzer, so far nothing has shown up yet.
fitzgen updated PR #6749.
fitzgen requested elliottt for a review on PR #6749.
elliottt submitted PR review:
Looks great!
elliottt submitted PR review:
Looks great!
elliottt created PR review comment:
Was this test new since the aarch64 pr landed?
elliottt created PR review comment:
for i in (0..new_stack_words).rev() {
fitzgen created PR review comment:
No, I just forgot to enable the tests in that PR.
fitzgen submitted PR review.
fitzgen updated PR #6749.
fitzgen has enabled auto merge for PR #6749.
fitzgen merged PR #6749.
Last updated: Dec 23 2024 at 12:05 UTC