sunfishcode opened PR #10657 from sunfishcode:sunfishcode/builtins to bytecodealliance:main:
With this patch, all emitted calls are Wasmtime-style builtins, rather than Cranelift-style libcalls. This ensures that all calls from Cranelift-generated code into Wasmtime host code use the same mechanism, and eliminates the relocation handling code for the libcall mechanism.
This is split out from #9093.
sunfishcode requested abrown for a review on PR #10657.
sunfishcode requested wasmtime-compiler-reviewers for a review on PR #10657.
sunfishcode requested pchickey for a review on PR #10657.
sunfishcode requested wasmtime-core-reviewers for a review on PR #10657.
sunfishcode requested wasmtime-default-reviewers for a review on PR #10657.
alexcrichton submitted PR review.
alexcrichton created PR review comment:
In combination with dropping these methods below, could this be changed to
unreachable()in the same manner asi8x16? Fixing this would otherwise require updates to the macro to have a separate "finger" for float registers as it has for integer registers right now, so this'll need more updates if floats are ever passed here.
alexcrichton created PR review comment:
This is technically incorrect because floats are stored in
FRegVal, notXRegVal. I see this is added here for the various macro machinations for crossing between Pulley and the host. Could this be dropped coupled with the suggestion above as well?
sunfishcode updated PR #10657.
sunfishcode updated PR #10657.
sunfishcode created PR review comment:
Done.
sunfishcode submitted PR review.
sunfishcode submitted PR review.
sunfishcode created PR review comment:
Done.
sunfishcode commented on PR #10657:
The failure was:
8.538 E: Failed to fetch http://security.ubuntu.com/ubuntu/pool/main/liba/libarchive/libarchive13_3.6.0-1ubuntu1.4_amd64.deb 404 Not Found [IP: 91.189.91.83 80]
8.538 E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?so, trying again.
sunfishcode updated PR #10657.
alexcrichton submitted PR review.
alexcrichton created PR review comment:
Mind also deleting these setters too? (fine to do in a follow-up too)
sunfishcode has enabled auto merge for PR #10657.
sunfishcode has disabled auto merge for PR #10657.
sunfishcode updated PR #10657.
sunfishcode has enabled auto merge for PR #10657.
sunfishcode submitted PR review.
sunfishcode created PR review comment:
Done.
sunfishcode updated PR #10657.
sunfishcode updated PR #10657.
sunfishcode updated PR #10657.
sunfishcode updated PR #10657.
sunfishcode merged PR #10657.
Last updated: Dec 06 2025 at 06:05 UTC