Rollup of 3 pull requests#151911
Conversation
Remove unused method `DroplessArena::contains_slice` - This method was added for rust-lang#120128. - It became unused in rust-lang#136593. Checking whether a particular slice is within an arena is a bit of a sketchy operation, so if there's no pressing need for it then I think we're better off not having it lying around.
…r=jackh726 refactor: add an `enum DerefAdjustKind` in favor of `Option<OverloadedDeref>` Part of rust-lang#149130. r? jackh726
…matter-ice, r=Kivooeo Fix ICE when parsing frontmatter without newline Fixes rust-lang#151882 we can not add a normal test case for it: - no newline at the end of file, we can bypass this with change test file name with `ignore-tidy` - multiple errors in stderr, this conflicts with the previous bypass, seems we can not add multiple error annotations in one line anyway, I added a `run-make` test for it.
|
@bors r+ rollup=never p=5 |
|
The job Click to see the possible cause of the failure (guessed by this bot) |
This comment has been minimized.
This comment has been minimized.
|
📌 Perf builds for each rolled up PR:
previous master: 78865ca937 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 78865ca (parent) -> 8afe9ff (this PR) Test differencesShow 26 test diffsStage 1
Stage 2
Additionally, 22 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 8afe9ff1caa97654c31fb8c259dac9fdf67d6302 --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 (8afe9ff): 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)This benchmark run did not return any relevant results for this metric. CyclesResults (secondary -17.0%)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: 476.918s -> 476.06s (-0.18%) |
Successful merges:
DroplessArena::contains_slice#151908 (Remove unused methodDroplessArena::contains_slice)enum DerefAdjustKindin favor ofOption<OverloadedDeref>#151850 (refactor: add anenum DerefAdjustKindin favor ofOption<OverloadedDeref>)r? @ghost
Create a similar rollup