Skip to content

Rustup - #5296

Merged
RalfJung merged 6 commits into
rust-lang:masterfrom
RalfJung:rustup
Aug 29, 2026
Merged

Rustup#5296
RalfJung merged 6 commits into
rust-lang:masterfrom
RalfJung:rustup

Conversation

@RalfJung

Copy link
Copy Markdown
Member

No description provided.

bors and others added 6 commits August 28, 2026 08:06
…kang

Refactor the `#[allow(dead_code)]` propagation for impl items of traits



Extracted from rust-lang/rust#157885.

This PR does the refactor and corrects the previous implementation.

The following will fail currently ([play](https://play.rust-lang.org/?version=nightly&mode=debug&edition=2024&gist=3279902b2d2e6b0fe75c5af565b1cddd)):
```rust
#![deny(dead_code)]
#![deny(unfulfilled_lint_expectations)]

#[allow(dead_code)]
pub trait Tr {
    fn foo(&self);
}

struct Foo;

impl Tr for Foo {
    fn foo(&self) {
        bar();
    }
}

#[expect(dead_code)]
fn bar() {}

fn main() {}
```

After this PR, we could handle the `#[allow(dead_code)]` propagation correctly, and should get perf improvement.
Consolidate LLVM skip in check builds in bootstrap

Before it was distributed amongst two places, which made it harder to follow the logic, and the `RUST_CHECK` logic was also applied for *anything* being built, even though it is only relevant for rustc itself. This PR centralizes the handling, and also enables it for `doc` builds.

So `./x doc compiler` now doesn't require building LLVM 🎉

r? jieyouxu
This updates the rust-version file to 17fd5b8a37b6667b6cc137f3cc35f09759768a3b.
@rustbot rustbot added the S-waiting-on-review Status: Waiting for a review to complete label Aug 29, 2026
@RalfJung
RalfJung enabled auto-merge August 29, 2026 06:25
@RalfJung
RalfJung added this pull request to the merge queue Aug 29, 2026
Merged via the queue into rust-lang:master with commit 476d7fd Aug 29, 2026
14 checks passed
@RalfJung
RalfJung deleted the rustup branch August 29, 2026 07:26
@rustbot rustbot removed the S-waiting-on-review Status: Waiting for a review to complete label Aug 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants