Rollup of 5 pull requests#155267
Conversation
This implements `-Z allow-partial-mitigations` as an unstable option, currently with support for control-flow-guard and stack-protector. As a difference from the RFC, we have `-Z allow-partial-mitigations=!foo` rather than `-Z deny-partial-mitigations=foo`, since I couldn't find an easy way to have an allow/deny pair of flags where the latter flag wins. To allow for stabilization, this is only enabled starting from the next edition. Maybe a better policy is possible (bikeshed).
…ments For the unbalanced backtick check. This fix is arguably a hack, however the original tidy check implementation is already based on heuristics and so are likewise fuzzy. This is probably fine in practice. Co-authored-by: Waffle Lapkin <waffle.lapkin@gmail.com>
…s, r=rcvalle Implement `-Z allow-partial-mitigations` (RFC 3855) This implements `-Z allow-partial-mitigations` as an unstable option, currently with support for control-flow-guard and stack-protector. As a difference from the RFC, we have `-Z allow-partial-mitigations=!foo` rather than `-Z deny-partial-mitigations=foo`, since I couldn't find an easy way to have an allow/deny pair of flags where the latter flag wins. To allow for stabilization, this is only enabled starting from the next edition. Maybe a better policy is possible (bikeshed). r? @rcvalle
…ter-cast-suggestion, r=davidtwco Refactor: simplify report_selection_error Split out `suggest_cast_to_fn_pointer` for better readability.
…s, r=jackh726 Trait aliases: Also imply default trait bounds on type params other than `Self` Trait aliases already correctly imply default trait bounds on `Self` type params. However, due to an oversight, they didn't do that for normal type params. Fixes rust-lang#152687.
rustdoc: dep-info for standalone markdown inputs Part of rust-lang#146220 (comment) r? @fmease
…=WaffleLapkin tidy: handle `#[cfg_attr(bootstrap, doc = "...")]` in `compiler/` comments For the unbalanced backtick check that Waffle ran into in rust-lang#154887. This PR cherry-picks Waffle's tidy patch in that PR (and adds some explaining comments) even though I'd say this is somewhat hacky[^1]. But since the original tidy check implementation is already based on heuristics and so are likewise fuzzy, this is probably fine in practice for most cases. [^1]: There can be false positives/negatives like having both fragments `cfg_attr` and `doc` inside a string literal, but I would expect that to be very very niche, so it's "good enough". (I wanted to write a regression test, but this check needs some restructuring to make it more easy to test that I don't want to bundle in this PR.) r? wafflelapkin (or bootstrap/compiler)
|
@bors r+ rollup=never p=5 |
This comment has been minimized.
This comment has been minimized.
|
📌 Perf builds for each rolled up PR:
previous master: 338dff3e3a 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 338dff3 (parent) -> 0006519 (this PR) Test differencesShow 64 test diffsStage 1
Stage 2
Additionally, 12 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 00065197834fd059fdaea30df5b6d1bf9791e539 --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 (0006519): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)Results (primary 2.1%, secondary -1.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -2.3%, secondary -3.6%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (secondary 0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 489.33s -> 491.89s (0.52%) |
Successful merges:
-Z allow-partial-mitigations(RFC 3855) #149357 (Implement-Z allow-partial-mitigations(RFC 3855))Self#152688 (Trait aliases: Also imply default trait bounds on type params other thanSelf)#[cfg_attr(bootstrap, doc = "...")]incompiler/comments #155195 (tidy: handle#[cfg_attr(bootstrap, doc = "...")]incompiler/comments)r? @ghost
Create a similar rollup