Skip to content

c-variadic: implement va_arg for wasm64#152594

Merged
rust-bors[bot] merged 1 commit intorust-lang:mainfrom
folkertdev:va-list-wasm64
Feb 14, 2026
Merged

c-variadic: implement va_arg for wasm64#152594
rust-bors[bot] merged 1 commit intorust-lang:mainfrom
folkertdev:va-list-wasm64

Conversation

@folkertdev
Copy link
Contributor

tracking issue: #44930

A LLVM maintainer of the wasm backend confirmed that the behavior on wasm64 is intented: the slot size is 4 on both wasm32 and wasm64. llvm/llvm-project#173580 (comment)

r? workingjubilee
cc @alexcrichton as target maintainer

@folkertdev folkertdev added the F-c_variadic `#![feature(c_variadic)]` label Feb 13, 2026
@rustbot rustbot added A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Feb 13, 2026
@rustbot
Copy link
Collaborator

rustbot commented Feb 13, 2026

workingjubilee is currently at their maximum review capacity.
They may take a while to respond.

@alexcrichton
Copy link
Member

@bors: r+

Thanks again for pushing on this! Agreed this looks reasonable given Derek's comment

@rust-bors
Copy link
Contributor

rust-bors bot commented Feb 14, 2026

Unknown command ":". Run @bors help to see available commands.

@alexcrichton
Copy link
Member

@bors r+

@rust-bors
Copy link
Contributor

rust-bors bot commented Feb 14, 2026

📌 Commit cc9e8c9 has been approved by alexcrichton

It is now in the queue for this repository.

@rust-bors rust-bors bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 14, 2026
rust-bors bot pushed a commit that referenced this pull request Feb 14, 2026
Rollup of 11 pull requests

Successful merges:

 - #151998 (Set hidden visibility on naked functions in compiler-builtins)
 - #149460 (rustdoc: sort stable items first)
 - #152076 (Feed `ErrorGuaranteed` from late lifetime resolution errors through to bound variable resolution)
 - #152471 (improve associated-type suggestions from bounds)
 - #152573 (move `escape_symbol_name` to `cg_ssa`)
 - #152594 (c-variadic: implement `va_arg` for `wasm64`)
 - #151386 (rustdoc: more js cleanup)
 - #152567 (nix-dev-shell: fix a typo)
 - #152568 (Port `#[lang]` and `#[panic_handler]` to the new attribute parsers)
 - #152575 (layout_of unexpected rigid alias delayed bug)
 - #152587 (A couple of tiny polonius things)
@rust-bors rust-bors bot merged commit 61dee57 into rust-lang:main Feb 14, 2026
11 checks passed
@rustbot rustbot added this to the 1.95.0 milestone Feb 14, 2026
rust-timer added a commit that referenced this pull request Feb 14, 2026
Rollup merge of #152594 - folkertdev:va-list-wasm64, r=alexcrichton

c-variadic: implement `va_arg` for `wasm64`

tracking issue: #44930

A LLVM maintainer of the wasm backend confirmed that the behavior on wasm64 is intented: the slot size is 4 on both wasm32 and wasm64. llvm/llvm-project#173580 (comment)

r? workingjubilee
cc @alexcrichton as target maintainer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. F-c_variadic `#![feature(c_variadic)]` S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants