Skip to content

va_arg: pass in TyAndLayout - #163021

Open
folkertdev wants to merge 1 commit into
rust-lang:mainfrom
folkertdev:va-arg-ty-and-layout
Open

folkertdev wants to merge 1 commit into
rust-lang:mainfrom
folkertdev:va-arg-ty-and-layout

Conversation

@folkertdev

Copy link
Copy Markdown
Contributor

Just a refactor, no functional changes. It is weird I did not spot this before, it cleans up the code a bunch.

weird I did not spot this before, it cleans up the code a bunch
@rustbot rustbot added A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Sep 19, 2026
@folkertdev
folkertdev marked this pull request as ready for review September 19, 2026 17:02
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Sep 19, 2026
@rustbot

rustbot commented Sep 19, 2026

Copy link
Copy Markdown
Collaborator

r? @adwinwhite

rustbot has assigned @adwinwhite.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 77 candidates
  • Random selection from 18 candidates

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. 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants