Rollup of 9 pull requests#153456
Conversation
It has no effect. Presumably at some point in the past there were generics involved here?
When running the `install src` command I'm seeing failures as the `builder.doc_out(host)` directory does not exist. This is because `match_paths_to_steps_and_run()` doesn't actually build any documentation as the `paths.is_empty()` causes an early return. This results in install failures as the `*/doc` src directory doesn't exist. This patch ensures that the builder.doc_out(host) directory exists. This fixes installing the Rust source when `build.docs = false`. This fixes installing the Rust source code in OpenEmbedded. Signed-off-by: Alistair Francis <alistair.francis@wdc.com> Signed-off-by: Jieyou Xu <jieyouxu@outlook.com>
Methods that perform internal iteration are typically named `for_each`.
…Amanieu
constify `Vec::{into, from}_raw_parts{_in|_alloc}`
due to Vec::drop not being const this is kinda necessary to make use of `const_heap`.
…risDenton Use shlex instead of shell-words In rust-lang#152712, the [`shell-words`] crate was introduced as a new dependency of `rustc_llvm` in order for its build script to parse the output of `llvm-config --quote-paths` and thereby handle paths containing whitespace; however, as [noted by bjorn3], that build script already transitively depends upon the [`shlex`] crate (via the [`cc`] crate) which provides similar functionality. This patch is based off (the already-approved) rust-lang#153419, which would otherwise conflict. [`cc`]: https://crates.io/crates/cc [noted by bjorn3]: rust-lang#152712 (comment) [`shell-words`]: https://crates.io/crates/shell-words [`shlex`]: https://crates.io/crates/shlex r? ChrisDenton
…p, r=jieyouxu bootstrap: minimal fix for ./x install src with build.docs = false `run_default_doc_steps()` is called to ensure the documentation is built by `Docs::run()` and it should build the documentation if it isn't already built. When running the `install src` command I'm seeing failures as the `builder.doc_out(host)` directory does not exist. This is because `match_paths_to_steps_and_run()` doesn't actually build any documentation as the `paths.is_empty()` causes an early return. This results in install failures as the `*/doc` src directory doesn't exist. This patch passes the paths to `run_step_descriptions()` when building documentation to ensure it is correctly built. This fixes installing the Rust source code in OpenEmbedded.
Make `const_lit_matches_ty` check literal suffixes for exact type match `const_lit_matches_ty` ignored literal suffixes. This let the `try_lower_anon_const_lit` fast path produce a mistyped `ty::Const::Value`, bypassing the type mismatch error that typeck would otherwise report.
Rename `QueryCache::iter` to `for_each` Methods that perform internal iteration are typically named `for_each`.
…r=petrochenkov Minor query cleanups Details in individual commits. r? @petrochenkov
Add a comment explaining the 'tcx lifetime. I wrote this after thinking about where the `'tcx` lifetime actually comes from. r? @bjorn3
…e, r=petrochenkov Fix obtaining def_id from unresolved segment This PR fixes ICE when trying to obtain `def_id` from an unresolved segment, part of rust-lang#118212, fixes rust-lang#153389. r? @petrochenkov
Fix ICEs due to incomplete typechecking on struct literals with syntax errors. Fixes rust-lang#153388. Followup to rust-lang#153227. Today I have learned that when we don’t emit a diagnostic *specifically from typeck*, we need to call `self.infcx.set_tainted_by_errors()` to signal that the type checking is incomplete despite the lack of error. r? fmease
|
@bors r+ rollup=never p=5 |
|
Trying commonly failed jobs |
|
⌛ Trying commit 11c4bea with merge 2bfe6e5… To cancel the try build, run the command Workflow: https://github.com/rust-lang/rust/actions/runs/22731374655 |
Rollup of 9 pull requests try-job: test-various try-job: x86_64-gnu-aux try-job: x86_64-gnu-llvm-21-3 try-job: x86_64-msvc-1
This comment has been minimized.
This comment has been minimized.
|
📌 Perf builds for each rolled up PR:
previous master: 64b72a1fa5 In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
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 64b72a1 (parent) -> 69370dc (this PR) Test differencesShow 392 test diffsStage 1
Stage 2
Additionally, 388 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 69370dc4a8862b8401615a2a7b950704ba66c495 --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 |
|
Finished benchmarking commit (69370dc): comparison URL. Overall result: ❌ regressions - no action needed@rustbot label: -perf-regression Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (primary -1.8%, secondary -1.5%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 2.9%, secondary 4.6%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 479.735s -> 482.035s (0.48%) |
|
💥 Test timed out after |
Successful merges:
Vec::{into, from}_raw_parts{_in|_alloc}#153399 (constifyVec::{into, from}_raw_parts{_in|_alloc})const_lit_matches_tycheck literal suffixes for exact type match #152906 (Makeconst_lit_matches_tycheck literal suffixes for exact type match)QueryCache::itertofor_each#153378 (RenameQueryCache::itertofor_each)r? @ghost
Create a similar rollup