Repository navigation
Rollup of 5 pull requests - #163842
Rollup of 5 pull requests#163842
Conversation
* fix visible for leak check in fast path * add comment * fix typo Co-authored-by: Rémy Rakic <remy.rakic+github@gmail.com>
Co-authored-by: Clar Fon <15850505+clarfonthey@users.noreply.github.com>
[rustdoc] Fix how `Deref` items is handled. Fixes rust-lang#160236. This PR fixes a few things around how we handle `Deref`: * Since nothing except for methods is callable through a `Deref`, only methods should be kept * If the `Deref::Target` is a type implementing `Copy`, then methods taking `self` work and should be displayed. <del>During the `clean` pass, I store the item `DefId` on which `Deref` is implemented in a `DefIdSet` if the `Deref::Target` implements `Copy`. Then during rendering, we check if the "parent item" is in in the `DefIdSet`, and if so, we keep methods with `self`.</del> <del>Now you might wonder why the item and not the derefed item. It's because the `DefId` we get from the computed type doesn't match the `DefId` of the actual type (that's where I spent most of my time, finding an ID (`DefId`/`ItemId`) I can use as key T_T).</del> It computes in `html/render` if the `Deref::Target` item is copy before rendering it. So to resume: * `&self` is always kept. * `&mut self` is kept if `DerefMut` is implemented (nothing changed there). * `self` is kept only if `Deref::Target` is `Copy`. * Everything else disappears as they're not callable through `Deref`. r? @camelid
…ath, r=BoxyUwU fix `VisibleForLeakCheck` in `RegionOutlives` fast path This causes the new solver to more closely match the old solver. `-Zdisable-fast-paths` already causes this to fail on nightly. This is a small bug in the impl of rust-lang#155749. We should have probably FCP'd that change as it was a breaking change to coherence. I forgot to do so after mentioning its impact on coherence in rust-lang#155749 (review). This PR causes the following snippet to no longer compile: ```rust trait MayRequireStatic {} // `for<'a> &'a i32: MayRequireStatic` does not hold impl<'a: 'static> MayRequireStatic for &'a i32 {} impl<'a> MayRequireStatic for &'a u32 {} trait Trait {} impl Trait for i32 {} impl<T> Trait for T where for<'a> &'a T: MayRequireStatic {} fn is_trait<T: Trait>() {} fn main() { is_trait::<i32>(); is_trait::<u32>(); } ``` This feels very artificial to me. I guess let's still do a quick FCP for this breakage. no clue why we're missing a span with the old solver here 🤣 r? BoxyUwU
…, r=JonathanBrouwer Move #[doc(inline)] and #[doc(no_inline)] validation into rustc_attr_parsing Part of rust-lang#153101 This migrates `#[doc(inline)]` and `#[doc(no_inline)]` validation from `rustc_passes` into `DocParser` in `rustc_attr_parsing`. It checks for conflicting inlining attributes and non-use targets, and tweaks rustdoc's early abort so lowering errors don't suppress other attribute diagnostics, also added new UI tests in doc-inline-validation.rs.
…n, r=notriddle When documenting without a specified `--edition`, emit a message Follow-up of rust-lang#158102. cc @estebank r? @notriddle
…they Add some comments and docs related to `AllocatorNightly` Follow-up to rust-lang#163590 (comment) and [this zulip thread](https://rust-lang.zulipchat.com/#narrow/channel/197181-t-libs.2Fwg-allocators/topic/.60AllocatorNightly.60.20auto-trait.3F/near/629283171)
This comment has been minimized.
This comment has been minimized.
…uwer Rollup of 5 pull requests Successful merges: - #160915 ([rustdoc] Fix how `Deref` items is handled.) - #163267 (fix `VisibleForLeakCheck` in `RegionOutlives` fast path) - #163673 (Move #[doc(inline)] and #[doc(no_inline)] validation into rustc_attr_parsing) - #163795 (When documenting without a specified `--edition`, emit a message) - #163814 (Add some comments and docs related to `AllocatorNightly`)
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
💔 Test for 5a5c446 failed: CI. Failed job:
|
|
@bors retry |
This comment has been minimized.
This comment has been minimized.
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
GitHub claims this is resolved. https://www.githubstatus.com/incidents/3q1yb5m7ltvb @bors treeclosed- |
|
Tree is now open for merging. |
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 ea13733 (parent) -> db23a2d (this PR) Test differencesShow 855 test diffsStage 1
Stage 2
Additionally, 836 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 db23a2d392783030c008a5fafbe6cb139d1f7707 --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 (db23a2d): comparison URL. Overall result: ❌ regressions - please read:Our benchmarks found a performance regression caused by this PR. Next Steps:
@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.7%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -1.5%, secondary 3.7%)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: 496.563s -> 489.541s (-1.41%) |
|
📌 Perf builds for each rolled up PR:
parent commit: ea137335b7 In the case of a perf regression, run the following command with the SHAs of each PR you suspect might be the cause: |
|
I think this is the Deref rusdoc one, but let's try, this should be fast. @rust-timer triage all |
Running triage with 2 benchmarksTriage only executes the benchmarks on rollup members, that were changed significantly on the rollup.
#160915 9d02933 [rustdoc] Fix how
|
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
0.3% | [0.3%, 0.3%] | 1 |
| Regressions ❌ (secondary) |
1.2% | [1.2%, 1.2%] | 1 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
- | - | 0 |
| All ❌✅ (primary) | 0.3% | [0.3%, 0.3%] | 1 |
Max RSS (memory usage)
This perf run didn't have relevant results for this metric.
Cycles
Results (secondary 6.5%)
A less reliable metric. May be of interest, but not used to determine the overall result above.
| mean | range | count | |
|---|---|---|---|
| Regressions ❌ (primary) |
- | - | 0 |
| Regressions ❌ (secondary) |
6.5% | [6.5%, 6.5%] | 1 |
| Improvements ✅ (primary) |
- | - | 0 |
| Improvements ✅ (secondary) |
- | - | 0 |
| All ❌✅ (primary) | - | - | 0 |
Binary size
This perf run didn't have relevant results for this metric.
#163267 2199e4e fix VisibleForLeakCheck in RegionOutlives fast path
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
This perf run didn't have relevant results for this metric.
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
#163673 274395e Move #[doc(inline)] and #[doc(no_inline)] validation into rustc_attr_parsing
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
This perf run didn't have relevant results for this metric.
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
#163795 9588ea2 When documenting without a specified --edition, emit a message
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
This perf run didn't have relevant results for this metric.
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
#163814 998b4e3 Add some comments and docs related to AllocatorNightly
This perf run didn't have relevant results for the `instruction count` metric.
Instruction count
This perf run didn't have relevant results for this metric.
Max RSS (memory usage)
This perf run didn't have relevant results for this metric.
Cycles
This perf run didn't have relevant results for this metric.
Binary size
This perf run didn't have relevant results for this metric.
Successful merges:
Derefitems is handled. #160915 ([rustdoc] Fix howDerefitems is handled.)VisibleForLeakCheckinRegionOutlivesfast path #163267 (fixVisibleForLeakCheckinRegionOutlivesfast path)--edition, emit a message #163795 (When documenting without a specified--edition, emit a message)AllocatorNightly#163814 (Add some comments and docs related toAllocatorNightly)r? @ghost
Create a similar rollup