Skip to content

RustWrapper: Suppress getNextNonDebugInfoInstruction - #144352

Merged
bors merged 1 commit into
rust-lang:masterfrom
heiher:llvm-22
Jul 26, 2025
Merged

RustWrapper: Suppress getNextNonDebugInfoInstruction#144352
bors merged 1 commit into
rust-lang:masterfrom
heiher:llvm-22

Conversation

@heiher

@heiherheiher commented Jul 23, 2025

Copy link
Copy Markdown
Contributor

@rustbot

rustbot commented Jul 23, 2025

Copy link
Copy Markdown
Collaborator

r? @cuviper

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

@rustbotrustbot 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 Jul 23, 2025
@dianqkdianqk added the llvm-main Marks PRs that are making Rust work with LLVM main (this label is consumed by CI tooling) label Jul 23, 2025
dianqk
dianqk reviewed Jul 23, 2025
View reviewed changes
Comment threadcompiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp
dianqk
dianqk approved these changes Jul 23, 2025
View reviewed changes

@dianqkdianqk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thanks!
r? dianqk @bors r+ rollup=always

@rustbotrustbot assigned dianqk and unassigned cuviperJul 23, 2025
@cuviper

cuviper commented Jul 23, 2025

Copy link
Copy Markdown
Member

@bors r=dianqk rollup

@bors

bors commented Jul 23, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit 23fda60 has been approved by dianqk

It is now in the queue for this repository.

@borsbors 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 Jul 23, 2025
Kobzol added a commit to Kobzol/rust that referenced this pull request Jul 25, 2025
@Kobzol
Rollup merge ofrust-lang#144352- heiher:llvm-22, r=dianqk
77fffaa
RustWrapper: Suppress getNextNonDebugInfoInstruction
Link: llvm/llvm-project#144383
@KobzolKobzol mentioned this pull request Jul 25, 2025
Closed
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Jul 25, 2025
@GuillaumeGomez
Rollup merge ofrust-lang#144352- heiher:llvm-22, r=dianqk
82867b1
RustWrapper: Suppress getNextNonDebugInfoInstruction
Link: llvm/llvm-project#144383
@GuillaumeGomezGuillaumeGomez mentioned this pull request Jul 25, 2025
Closed
bors added a commit that referenced this pull request Jul 25, 2025
@bors
Auto merge of#144466- GuillaumeGomez:rollup-pvka4ir, r=GuillaumeGomez
0e2300b
Rollup of 14 pull requests
Successful merges:
- #144316 (bootstrap: Move musl-root fallback out of sanity check)
- #144352 (RustWrapper: Suppress getNextNonDebugInfoInstruction)
- #144356 (Add `ignore-backends` annotations in failing GCC backend ui tests)
- #144359 (add codegen test for variadics)
- #144376 (Suggest unwrapping when private method name is available in inner type)
- #144379 (test using multiple c-variadic ABIs in the same program)
- #144383 (disable cfg.has_reliable_f128 on amdgcn)
- #144412 (Small cleanup: Use LocalKey<Cell> methods more)
- #144421 (Call `is_parsed_attribute` rather than keeping track of a list of parsed attributes manually)
- #144422 (library/windows_targets: Fix macro expansion error in 'link' macro)
- #144424 (Allow setting `release-blog-post` label with rustbot)
- #144430 (tests: aarch64-outline-atomics: Remove hardcoded target)
- #144435 (rustc-dev-guide subtree update)
- #144445 (Fix `./x check bootstrap` (again))
r? `@ghost`
`@rustbot` modify labels: rollup
@tgross35tgross35 mentioned this pull request Jul 26, 2025
Merged
bors added a commit that referenced this pull request Jul 26, 2025
@bors
Auto merge of#144488- tgross35:rollup-vn0fpot, r=tgross35
051d0e8
Rollup of 9 pull requests
Successful merges:
- #144089 (Rehome 35 `tests/ui/issues/` tests to other subdirectories under `tests/ui/`)
- #144171 (pattern_analysis: add option to get a full set of witnesses)
- #144201 (Mention type that could be `Clone` but isn't in more cases)
- #144316 (bootstrap: Move musl-root fallback out of sanity check)
- #144339 (Enable dwarf-mixed-versions-lto.rs test on RISC-V (riscv64))
- #144341 (Enable const-vector.rs test on RISC-V (riscv64))
- #144352 (RustWrapper: Suppress getNextNonDebugInfoInstruction)
- #144356 (Add `ignore-backends` annotations in failing GCC backend ui tests)
- #144364 (Update `dlmalloc` dependency of libstd)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 2671afe into rust-lang:masterJul 26, 2025
11 checks passed
@rustbotrustbot added this to the 1.90.0 milestone Jul 26, 2025
rust-timer added a commit that referenced this pull request Jul 26, 2025
@rust-timer
Unrolled build for#144352
2cdd754
Rollup merge of #144352 - heiher:llvm-22, r=dianqk
RustWrapper: Suppress getNextNonDebugInfoInstruction
Link: llvm/llvm-project#144383
@heiher
heiher deleted the llvm-22 branch July 30, 2025 05:05
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dianqkdianqkdianqk approved these changes

Assignees

@dianqkdianqk

Labels

A-LLVMArea: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues.llvm-mainMarks PRs that are making Rust work with LLVM main (this label is consumed by CI tooling)S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Milestone

1.90.0

Development

Successfully merging this pull request may close these issues.

5 participants

@heiher@rustbot@cuviper@bors@dianqk