Skip to content

Commit 1afcdbc

Browse files
Merge pull request #1996 from Mark-Simulacrum/triage
Add triage for this week
2 parents b6c2f42 + 0eb45a6 commit 1afcdbc

File tree

1 file changed

+81
-0
lines changed

1 file changed

+81
-0
lines changed

triage/2024-10-14.md

Lines changed: 81 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,81 @@
1+
# 2024-10-14 Triage Log
2+
3+
No major changes this week.
4+
5+
Triage done by **@simulacrum**.
6+
Revision range: [e6c46db4..5ceb623a](https://perf.rust-lang.org/?start=e6c46db4e9fd11e3183c397a59d946731034ede6&end=5ceb623a4abd66e91e7959d25caaf0523f1a7f7c&absolute=false&stat=instructions%3Au)
7+
8+
**Summary**:
9+
10+
| (instructions:u) | mean | range | count |
11+
|:----------------------------------:|:-----:|:--------------:|:-----:|
12+
| Regressions ❌ <br /> (primary) | 0.2% | [0.2%, 0.2%] | 1 |
13+
| Regressions ❌ <br /> (secondary) | 0.5% | [0.2%, 2.9%] | 16 |
14+
| Improvements ✅ <br /> (primary) | -0.3% | [-0.4%, -0.2%] | 14 |
15+
| Improvements ✅ <br /> (secondary) | -0.6% | [-1.7%, -0.1%] | 32 |
16+
| All ❌✅ (primary) | -0.2% | [-0.4%, 0.2%] | 15 |
17+
18+
19+
1 Regression, 1 Improvement, 2 Mixed; 2 of them in rollups
20+
36 artifact comparisons made in total
21+
22+
#### Regressions
23+
24+
Introduce SolverRelating type relation to the new solver [#131263](https://github.com/rust-lang/rust/pull/131263) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=4cc494bbfe9911d24f3ee521f98d5c6bb7e3ffe8&end=8d94e06ec9758b5c03ea77bb5dab22a1a76bc261&stat=instructions:u)
25+
26+
| (instructions:u) | mean | range | count |
27+
|:----------------------------------:|:----:|:------------:|:-----:|
28+
| Regressions ❌ <br /> (primary) | 0.2% | [0.2%, 0.2%] | 1 |
29+
| Regressions ❌ <br /> (secondary) | 0.5% | [0.2%, 2.7%] | 9 |
30+
| Improvements ✅ <br /> (primary) | - | - | 0 |
31+
| Improvements ✅ <br /> (secondary) | - | - | 0 |
32+
| All ❌✅ (primary) | 0.2% | [0.2%, 0.2%] | 1 |
33+
34+
Genuine, if small, regressions on a number of benchmarks. Likely required as
35+
part of the broader move to the new solver.
36+
37+
#### Improvements
38+
39+
Rollup of 8 pull requests [#131628](https://github.com/rust-lang/rust/pull/131628) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=6b9676b45431a1e531b9c5f7bd289fc36a312749&end=ef4e8259b5016d85e261587b605028b2ff06c13d&stat=instructions:u)
40+
41+
| (instructions:u) | mean | range | count |
42+
|:----------------------------------:|:-----:|:--------------:|:-----:|
43+
| Regressions ❌ <br /> (primary) | - | - | 0 |
44+
| Regressions ❌ <br /> (secondary) | 0.5% | [0.3%, 0.6%] | 2 |
45+
| Improvements ✅ <br /> (primary) | -0.3% | [-0.3%, -0.2%] | 8 |
46+
| Improvements ✅ <br /> (secondary) | -0.6% | [-1.8%, -0.1%] | 31 |
47+
| All ❌✅ (primary) | -0.3% | [-0.3%, -0.2%] | 8 |
48+
49+
Seems most likely to be due to "Shallowly match opaque key in storage
50+
[#131599](https://github.com/rust-lang/rust/pull/131599)" but not confirming
51+
since this is an improvement.
52+
53+
#### Mixed
54+
55+
Rollup of 7 pull requests [#131581](https://github.com/rust-lang/rust/pull/131581) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=fb20e4d3b96d1de459d086980a8b99d5060ad9fe&end=8f8bee4f60d9d3769f75c70d558c27a95761c554&stat=instructions:u)
56+
57+
| (instructions:u) | mean | range | count |
58+
|:----------------------------------:|:-----:|:--------------:|:-----:|
59+
| Regressions ❌ <br /> (primary) | - | - | 0 |
60+
| Regressions ❌ <br /> (secondary) | 0.4% | [0.3%, 0.6%] | 6 |
61+
| Improvements ✅ <br /> (primary) | -0.3% | [-1.0%, -0.2%] | 7 |
62+
| Improvements ✅ <br /> (secondary) | -2.5% | [-2.5%, -2.5%] | 1 |
63+
| All ❌✅ (primary) | -0.3% | [-1.0%, -0.2%] | 7 |
64+
65+
Possibly spurious (and/or bi-modality) regression confined to tt-muncher
66+
(though across multiple scenarios). No need for further investigation,
67+
particular given a rollup.
68+
69+
Add `#[track_caller]` to allocating methods of `Vec` & `VecDeque` [#126557](https://github.com/rust-lang/rust/pull/126557) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=5ceb623a4abd66e91e7959d25caaf0523f1a7f7c&end=f6648f252a05a0a46c865d7ec836b46290613bf9&stat=instructions:u)
70+
71+
| (instructions:u) | mean | range | count |
72+
|:----------------------------------:|:-----:|:--------------:|:-----:|
73+
| Regressions ❌ <br /> (primary) | 0.3% | [0.0%, 0.5%] | 26 |
74+
| Regressions ❌ <br /> (secondary) | - | - | 0 |
75+
| Improvements ✅ <br /> (primary) | - | - | 0 |
76+
| Improvements ✅ <br /> (secondary) | -0.2% | [-0.2%, -0.1%] | 4 |
77+
| All ❌✅ (primary) | 0.3% | [0.0%, 0.5%] | 26 |
78+
79+
Seems to be accepted by T-libs
80+
(https://github.com/rust-lang/rust/pull/126557#issuecomment-2329482185).
81+
An expected regression pre-merge.

0 commit comments

Comments
 (0)