Skip to content

Vendor __rust_probestack for Rust 1.89+#9

Open
deuszx wants to merge 1 commit intolinera-wasmerfrom
fix-rust-probestack
Open

Vendor __rust_probestack for Rust 1.89+#9
deuszx wants to merge 1 commit intolinera-wasmerfrom
fix-rust-probestack

Conversation

@deuszx
Copy link

@deuszx deuszx commented Mar 2, 2026

Starting with Rust 1.89, the __rust_probestack symbol from compiler-builtins is name-mangled and no longer exported, causing linker failures when building wasmer-vm on x86/x86_64.

Conditionally vendor the probestack assembly from compiler-builtins when building with Rust >= 1.89, using rustversion for detection. On older Rust versions the behavior is unchanged.

Cherry-picked from upstream wasmerio/wasmer@fc021be5c6f with a fix to use the old cargo: build script syntax for MSRV compatibility.

Starting with Rust 1.89, the __rust_probestack symbol from
compiler-builtins is name-mangled and no longer exported, causing
linker failures when building wasmer-vm on x86/x86_64.

Conditionally vendor the probestack assembly from compiler-builtins
when building with Rust >= 1.89, using rustversion for detection.
On older Rust versions the behavior is unchanged.

Cherry-picked from upstream wasmerio/wasmer@fc021be5c6f with a
fix to use the old cargo: build script syntax for MSRV compatibility.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@deuszx deuszx requested a review from Twey March 2, 2026 08:35
Copy link

@Twey Twey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM modulo tests

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure I understand why this is in a build script that just renames the cfg. Could it not be inlined?

@deuszx
Copy link
Author

deuszx commented Mar 10, 2026

This CI has been failing since the beginning (see the time when we released 4.4.X for linera #8)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants