Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

llvm-wrapper: adapt for LLVM API changes, second try #130509

Merged
merged 1 commit into from
Sep 18, 2024
Merged

llvm-wrapper: adapt for LLVM API changes, second try #130509

merged 1 commit into from
Sep 18, 2024

Conversation

krasimirgg
Copy link
Contributor

This is a re-work of #129749 after LLVM brought back the APIs used by rust.

No functional changes intended.

@rustbot label: +llvm-main
r? @nikic
cc: @tmandry

@rustbot rustbot added 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. llvm-main Marks PRs that are making Rust work with LLVM main (this label is consumed by CI tooling) labels Sep 18, 2024
@krasimirgg krasimirgg marked this pull request as ready for review September 18, 2024 14:08
@nikic
Copy link
Contributor

nikic commented Sep 18, 2024

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Sep 18, 2024

📌 Commit 3a35288 has been approved by nikic

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 Sep 18, 2024
@matthiaskrgr matthiaskrgr mentioned this pull request Sep 18, 2024
Merged
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 18, 2024
@bors
Auto merge of rust-lang#130519 - matthiaskrgr:rollup-l1hok4x, r=matth…
f79a912 
…iaskrgr

Rollup of 5 pull requests

Successful merges:

 - rust-lang#130457 (Cleanup codegen traits)
 - rust-lang#130471 (Add zlib to musl dist image so rust-lld will support zlib compression for debug info there.)
 - rust-lang#130507 (Improve handling of raw-idents in check-cfg)
 - rust-lang#130509 (llvm-wrapper: adapt for LLVM API changes, second try)
 - rust-lang#130510 (doc: the source of `LetStmt` can also be `AssignDesugar`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 48b90aa into rust-lang:master Sep 18, 2024
6 checks passed
@rustbot rustbot added this to the 1.83.0 milestone Sep 18, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Sep 18, 2024
@rust-timer
Unrolled build for rust-lang#130509
4d172b3 
Rollup merge of rust-lang#130509 - krasimirgg:llvm-20-2, r=nikic

llvm-wrapper: adapt for LLVM API changes, second try

This is a re-work of rust-lang#129749 after LLVM brought back the APIs used by rust.

No functional changes intended.

`@rustbot` label: +llvm-main
r? `@nikic`
cc: `@tmandry`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nikic nikic

Labels
llvm-main Marks PRs that are making Rust work with LLVM main (this label is consumed by CI tooling) 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
Milestone
1.83.0
Development

Successfully merging this pull request may close these issues.
4 participants
@krasimirgg @nikic @bors @rustbot