-
Notifications
You must be signed in to change notification settings - Fork 14k
Rollup of 6 pull requests #149276
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.
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
Rollup of 6 pull requests #149276
Conversation
This fixes a limitation that prevented me from adding it to Cargo. The rust compiler-docs bundle is built by running multiple `cargo` commands in succession, and I want to support that, so I'm stuck putting the doc parts all in one directory, so that subsequent `cargo` runs can pick up the previous runs' data. It's less clean, but it should still be usable in hermetic build environments if you give every crate its own directory (which you needed to do before, oddly enough).
This allows LTO to work when compiling jemalloc, which should yield a small performance boost, and makes miri's behaviour here match clippy and rustdoc.
rustdoc: make mergeable crate info more usable Part of rust-lang#130676 Adds documentation and a feature change aimed at making this usable without breaking backwards compat. cc ``@EtomicBomb``
Add suggest alternatives for Out-of-range \x escapes Fixes rust-lang#148917 seems add two notes seems better. r? `@scottmcm`
…amofek,lolbinarycat [rustdoc] Make more functions return `fmt::Result` and reduce number of `.unwrap()` calls Following our discussion in rust-lang#149028 (comment), this PR makes more function return `fmt::Result`, allowing to use `?` a lot more, and also reducing number of `.unwrap()` calls. r? `@lolbinarycat`
miri: use `tikv-jemalloc-sys` from sysroot This allows LTO to work when compiling jemalloc (which is currently broken due to rust-lang/cc-rs#1613), which should yield a small performance boost, and makes Miri's behaviour here match Clippy and Rustdoc. Follow-up to rust-lang#148925 / rust-lang#146627 after discussion in rust-lang#148925 (review). r? RalfJung
Use `let...else` consistently in user-facing diagnostics resolve: rust-lang#148772 Standardize `let...else` terminology in user-facing diagnostics. Goal: Consistently use `let...else` in all error/warning messages. Internal technical names (`let-else`) in comments and documentation remain unchanged. cc ``@carols10cents,`` ``@jieyouxu``
Fix missing double-quote in `std::env::consts::OS` values Noticed this one while reading the docs. Hopefully it's not too small change.
|
@bors r+ rollup=never p=5 |
|
☀️ Test successful - checks-actions |
|
📌 Perf builds for each rolled up PR:
previous master: b64df9d101 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 b64df9d (parent) -> c871d09 (this PR) Test differencesShow 7 test diffsStage 1
Stage 2
Additionally, 2 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 c871d09d1cc32a649f4c5177bb819646260ed120 --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 (c871d09): 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 1.4%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary -5.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary 0.0%, secondary 0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 470.372s -> 469.468s (-0.19%) |
Successful merges:
fmt::Resultand reduce number of.unwrap()calls #149208 ([rustdoc] Make more functions returnfmt::Resultand reduce number of.unwrap()calls)tikv-jemalloc-sysfrom sysroot #149252 (miri: usetikv-jemalloc-sysfrom sysroot)let...elseconsistently in user-facing diagnostics #149255 (Uselet...elseconsistently in user-facing diagnostics)std::env::consts::OSvalues #149275 (Fix missing double-quote instd::env::consts::OSvalues)r? @ghost
@rustbot modify labels: rollup
Create a similar rollup