Rollup of 6 pull requests - #163754
Rollup of 6 pull requests#163754
Conversation
Fix test-tidy review changes review changes review changes fix llvm build ignore attribute documentation example fix tidy use no_run instead of ignore
…r, r=mejrs FCW for `#[panic_handler]` on `unsafe fn`. Mitigates rust-lang#162967. A function annotated with `#[panic_handler]` can be called by the compiler from anywhere. So, such functions must not have any safety preconditions. A [github search](https://github.com/search?q=language%3Arust+%2F%23%5C%5Bpanic_handler%5C%5D%5Cnunsafe%2F&type=code) shows many crates that would run into this, so a hard error seems infeasible. If needed, I can modify the code in order to run crater to check how much code would be flagged by this FCW. ~~I have not yet created a proper tracking issue for the FCW. If this PR seems like the right direction, I will do so before merging.~~ I've created a tracking issue for this FCW at rust-lang#163263 An LLM pointed me towards `check_panic_info_fn` and `emit_node_span_lint` (which I verified to be right). However, this PR is otherwise written manually.
…ttribute, r=bushrat011899,JonathanBrouwer Add documentation for the `no_main` and `repr` attributes Part of rust-lang#157604. This PR documents `no_main` and `repr` attributes in `library/core/src/attribute_docs.rs` with some examples. Tested with: `./x doc`
…opt, r=Kobzol Add `--frontend-threads` option to `./x perf` This just adds an option to pass through the `--frontend-threads` option to `rustc-perf` through `./x perf` (which option is documented in the [`rustc-perf` repo](https://github.com/rust-lang/rustc-perf/blob/main/collector/README.md#benchmarking-options). Before this, `./x perf` had no way of telling `rustc-perf` how many frontend threads to use, so it always defaulted to 1. This made it a bit more difficult to test perf-related chnages to the parallel frontend. I tested this option locally and it seems to work perfectly fine.
…fonthey c_str_alloc_error test: mention why this is mostly Miri-only This was explained at the top of the file but I missed it there. Seems worth repeating at the attribute? Or am I just too blind? Cc @bjorn3 -- is it expected that `#[global_allocator]` does not work in alloctests?
…itor Stabilize `CStr::display` Passed FCP in rust-lang#139984. (closes rust-lang#139984) cc rust-lang#163710
…enyukang [tiny] Remove useless `.into()` calls
This comment has been minimized.
This comment has been minimized.
Rollup of 6 pull requests try-job: dist-various-1 try-job: test-various try-job: test-x86_64-gnu-aux try-job: test-x86_64-msvc-1 try-job: test-aarch64-apple-1 try-job: test-aarch64-apple-2 try-job: test-x86_64-mingw-1 try-job: test-i686-msvc try-job: test-armhf-gnu
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 a639ea0 (parent) -> f48b3e6 (this PR) Test differencesShow 131 test diffsStage 1
Stage 2
Additionally, 128 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 f48b3e61eeece268609bd920fcca1ded047db7ca --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 (f48b3e6): comparison URL. Overall result: ❌✅ regressions and improvements - 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 (secondary 1.4%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary -0.9%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: 489.631s -> 490.122s (0.10%) |
|
📌 Perf builds for each rolled up PR:
parent commit: a639ea0890 In the case of a perf regression, run the following command with the SHAs of each PR you suspect might be the cause: |
Successful merges:
#[panic_handler]onunsafe fn. #162974 (FCW for#[panic_handler]onunsafe fn.)no_mainandreprattributes #163627 (Add documentation for theno_mainandreprattributes)--frontend-threadsoption to./x perf#163671 (Add--frontend-threadsoption to./x perf)CStr::display#163711 (StabilizeCStr::display).into()calls #163723 ([tiny] Remove useless.into()calls)r? @ghost
Create a similar rollup