Rollup of 10 pull requests#154466
Conversation
Tests that dead code warnings work correctly inside const _ blocks. Issue was already fixed, adding regression test as requested.
This commit turns "Actual" into "Expected" for the following problem: Steps: 1. Do some quick and experimental local code changes in the compiler code. 2. Run `./x build`. 3. Observe `error: warnings are denied by `build.warnings` configuration`. 4. Decide to not care and want to allow warnings. 5. Search for `build.warnings` in **bootstrap.example.toml** Actual: No hits. You get frustrated because didn't learn how to allow warnings. Expected: You get a hit and can easily see how to allow warnings.
Fixes a rustfmt bug for `#![feature(fn_delegation)]`
…arser, r=clubby789 Unstable book options parser Parses the `options!` macro in `compiler/rustc_session/src/options.rs` directly to extract the unstable (-Z) compiler flag names and descriptions to generate documentation for the unstable book. I took notice from the previous attempt which ran `rustc -Zhelp` and parsed the output and used this approach that reads the source directly. Used claude for the tedious char by char parsing parts but verified the code, I hope that's ok!
…al, r=petrochenkov Use LocalDefId for more tcx method calls r? @petrochenkov Might bring perf improvements
…omment-bidi-help, r=JohnTitor Improve doc comment unicode guidance Fixes rust-lang#153096 This PR does not suggest HTML bidi markup, because doc comments are still Rust source first, not only rendered rustdoc output. HTML only helps in the rendered documentation rather than making them explicit in source.
…t-unrelated-leaf, r=JohnTitor Avoid ICE in explicit reference cast suggestion for unrelated leaf pr… Fixes rust-lang#154403 The explicit reference cast suggestion in was enabled based on `main_trait_predicate` being `From`/`TryFrom`, but it then extracted the source type from `leaf_trait_predicate`. That is only valid when the leaf obligation is also part of the `From`/`TryFrom` conversion family.
…alueFormat-mips, r=marcoieni Fix LegacyKeyValueFormat report from docker build: mips Part of rust-lang#152305 r? @marcoieni
…ngs, r=fmease Add test for issue rust-lang#101532: dead code warnings in const _ Closes rust-lang#101532 Adds a test for dead code warnings in `const _`. This was already fixed, just adding coverage to avoid regressions.
…oc-json-emit-conflict, r=aDotInTheVoid Rustdoc rejects html emits with json output Fixes rust-lang#154395
…ic-targets, r=jieyouxu bootstrap: `-Zjson-target-spec` for synthetic targets Fixes rust-lang#154340 Synthetic targets for mir-opt tests build a standard library using a `.json` target, which now requires `-Zjson-target-spec` r? @jieyouxu
bootstrap.example.toml: Hint how to allow `build.warnings` This commit turns **Actual** into **Expected** for the following problem: ### Steps 1. Do some quick and experimental local code changes in the compiler code. 2. Run `./x build`. 3. Observe failed build with _"error: warnings are denied by `build.warnings` configuration"_. 4. Decide to not care and want to allow warnings. 5. Search for `build.warnings` in **bootstrap.example.toml**. ### Actual No hits. You get frustrated because didn't learn how to allow warnings. (The reason `build.warnings` is not mentioned in **bootstrap.example.toml** is of course because it is an unstable cargo [feature](https://doc.rust-lang.org/cargo/reference/unstable.html#buildwarnings) and not controlled by bootstrap, but you don't know that yet.) ### Expected You get a hit and can easily see how to allow warnings.
fix: [rustfmt] prevent panic when rewritng associated item delegations Tracking issue: rust-lang#118212 rustfmt issue: rust-lang/rustfmt#6513 Fixes a rustfmt bug for `#![feature(fn_delegation)]`
|
@bors r+ p=5 |
This comment has been minimized.
This comment has been minimized.
Rollup of 10 pull requests Successful merges: - #154070 (Unstable book options parser) - #154371 (Use LocalDefId for more tcx method calls) - #154405 (Improve doc comment unicode guidance) - #154431 (Avoid ICE in explicit reference cast suggestion for unrelated leaf pr…) - #153528 (Fix LegacyKeyValueFormat report from docker build: mips) - #154246 (Add test for issue #101532: dead code warnings in const _) - #154421 (Rustdoc rejects html emits with json output) - #154428 (bootstrap: `-Zjson-target-spec` for synthetic targets) - #154437 (bootstrap.example.toml: Hint how to allow `build.warnings`) - #154454 (fix: [rustfmt] prevent panic when rewritng associated item delegations) Failed merges: - #154450 (Use the normal arg-parsing machinery for `-Zassert-incr-state`)
|
Very likely stuck, too :(; pesky |
|
@bors cancel |
|
Auto build was cancelled. Cancelled workflows: The next pull request likely to be tested is #154466. |
This comment has been minimized.
This comment has been minimized.
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing fda6d37 (parent) -> e161368 (this PR) Test differencesShow 43 test diffsStage 1
Stage 2
Additionally, 30 doctest diffs were found. These are ignored, as they are noisy. Job group index
Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard e1613686e0efc80a1a18b1263625450a8de3fb04 --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
📌 Perf builds for each rolled up PR:
previous master: fda6d37bb8 In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
|
Finished benchmarking commit (e161368): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results (primary -5.9%, secondary 6.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 485.109s -> 484.178s (-0.19%) |
Successful merges:
const _#101532: dead code warnings in const _)-Zjson-target-specfor synthetic targets #154428 (bootstrap:-Zjson-target-specfor synthetic targets)build.warnings#154437 (bootstrap.example.toml: Hint how to allowbuild.warnings)Failed merges:
-Zassert-incr-state#154450 (Use the normal arg-parsing machinery for-Zassert-incr-state)r? @ghost
Create a similar rollup