Skip to content

Conversation

@Zalathar
Copy link
Member

@Zalathar Zalathar commented Oct 7, 2025


This is another batch of adjustments to code that would otherwise interfere with moving FFI bindings out of rustc_codegen_llvm and into rustc_llvm.

@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 Oct 7, 2025
@rustbot
Copy link
Collaborator

rustbot commented Oct 7, 2025

r? @SparrowLii

rustbot has assigned @SparrowLii.
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

@rustbot

This comment has been minimized.

@bors
Copy link
Collaborator

bors commented Oct 8, 2025

☔ The latest upstream changes (presumably #147453) made this pull request unmergeable. Please resolve the merge conflicts.

@rustbot

This comment has been minimized.

@bors
Copy link
Collaborator

bors commented Oct 13, 2025

☔ The latest upstream changes (presumably #147629) made this pull request unmergeable. Please resolve the merge conflicts.

@rustbot

This comment has been minimized.

@wesleywiser
Copy link
Member

r? rust-lang/compiler

@rustbot rustbot assigned chenyukang and unassigned SparrowLii Dec 18, 2025
@chenyukang
Copy link
Member

@bors r=chenyukang

@bors
Copy link
Collaborator

bors commented Dec 19, 2025

📌 Commit 748621e has been approved by chenyukang

It is now in the queue for this repository.

@bors bors 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 Dec 19, 2025
@Zalathar
Copy link
Member Author

I haven't touched this in a while, so I'll rebase it to check that nothing unexpected has happened in the meantime.

@bors r-

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Dec 19, 2025
@rustbot
Copy link
Collaborator

rustbot commented Dec 19, 2025

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@Zalathar
Copy link
Member Author

Looks good still.

@bors r=chenyukang

@bors
Copy link
Collaborator

bors commented Dec 19, 2025

📌 Commit 735a980 has been approved by chenyukang

It is now in the queue for this repository.

@bors bors removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Dec 19, 2025
@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Dec 19, 2025
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Dec 19, 2025
cg_llvm: More preparation for moving FFI bindings to `rustc_llvm`

- Sequel to rust-lang#147327
- Motivated by rust-lang#142897

---

This is another batch of adjustments to code that would otherwise interfere with moving FFI bindings out of `rustc_codegen_llvm` and into `rustc_llvm`.
bors added a commit that referenced this pull request Dec 19, 2025
…uwer

Rollup of 4 pull requests

Successful merges:

 - #147430 (cg_llvm: More preparation for moving FFI bindings to `rustc_llvm`)
 - #149815 (Add regression test for #120189)
 - #149947 (add several older crashtests)
 - #150127 (Port `#[rustc_lint_untracked_query_information]` and `#[rustc_lint_diagnostics]` to using attribute parsers)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit that referenced this pull request Dec 19, 2025
Rollup of 3 pull requests

Successful merges:

 - #147430 (cg_llvm: More preparation for moving FFI bindings to `rustc_llvm`)
 - #149815 (Add regression test for #120189)
 - #150127 (Port `#[rustc_lint_untracked_query_information]` and `#[rustc_lint_diagnostics]` to using attribute parsers)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit that referenced this pull request Dec 19, 2025
Rollup of 3 pull requests

Successful merges:

 - #147430 (cg_llvm: More preparation for moving FFI bindings to `rustc_llvm`)
 - #149815 (Add regression test for #120189)
 - #150127 (Port `#[rustc_lint_untracked_query_information]` and `#[rustc_lint_diagnostics]` to using attribute parsers)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 91821aa into rust-lang:main Dec 19, 2025
11 checks passed
@rustbot rustbot added this to the 1.94.0 milestone Dec 19, 2025
rust-timer added a commit that referenced this pull request Dec 19, 2025
Rollup merge of #147430 - Zalathar:inherent, r=chenyukang

cg_llvm: More preparation for moving FFI bindings to `rustc_llvm`

- Sequel to #147327
- Motivated by #142897

---

This is another batch of adjustments to code that would otherwise interfere with moving FFI bindings out of `rustc_codegen_llvm` and into `rustc_llvm`.
@Zalathar Zalathar deleted the inherent branch December 19, 2025 11:50
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-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.

6 participants