Skip to content

Commit 9868f11

Browse files
Merge pull request #1649 from rylev/perf-2023-07-11
Add triage for 2023-07-11
2 parents e884e83 + 7690447 commit 9868f11

File tree

1 file changed

+118
-0
lines changed

1 file changed

+118
-0
lines changed

triage/2023-07-11.md

Lines changed: 118 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,118 @@
1+
# 2023-07-11 Triage Log
2+
3+
A very quiet week with nearly no changes in compiler performance.
4+
5+
Triage done by **@rylev**.
6+
Revision range: [52d8c49..1d4f5af](https://perf.rust-lang.org/?start=52d8c490a3aabe65cdd9f2d3aed95034dd5dbad7&end=1d4f5affbdee00c816f961c227c6b28a3e725ce6&absolute=false&stat=instructions%3Au)
7+
8+
**Summary**:
9+
10+
| (instructions:u) | mean | range | count |
11+
|:----------------------------------:|:-----:|:--------------:|:-----:|
12+
| Regressions ❌ <br /> (primary) | 1.1% | [0.7%, 1.7%] | 8 |
13+
| Regressions ❌ <br /> (secondary) | 2.1% | [0.4%, 3.7%] | 2 |
14+
| Improvements ✅ <br /> (primary) | -0.9% | [-1.2%, -0.5%] | 26 |
15+
| Improvements ✅ <br /> (secondary) | -1.2% | [-1.8%, -0.2%] | 16 |
16+
| All ❌✅ (primary) | -0.4% | [-1.2%, 1.7%] | 34 |
17+
18+
19+
4 Regressions, 2 Improvements, 2 Mixed; 1 of them in rollups
20+
51 artifact comparisons made in total
21+
22+
#### Regressions
23+
24+
Rollup of 8 pull requests [#113370](https://github.com/rust-lang/rust/pull/113370) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=e4cd1610067501fa4d347eba7b18f77137dbbf48&end=5dac6b320be868f898a3c753934eabc79ff2e406&stat=instructions:u)
25+
26+
| (instructions:u) | mean | range | count |
27+
|:----------------------------------:|:----:|:------------:|:-----:|
28+
| Regressions ❌ <br /> (primary) | 1.3% | [0.6%, 2.1%] | 11 |
29+
| Regressions ❌ <br /> (secondary) | 1.6% | [1.6%, 1.6%] | 1 |
30+
| Improvements ✅ <br /> (primary) | - | - | 0 |
31+
| Improvements ✅ <br /> (secondary) | - | - | 0 |
32+
| All ❌✅ (primary) | 1.3% | [0.6%, 2.1%] | 11 |
33+
- [#113320](https://github.com/rust-lang/rust/pull/113320) seemed to be the culprit but it's not clear why. Following up with the author.
34+
35+
Remove some unnecessary(?) normalization [#113348](https://github.com/rust-lang/rust/pull/113348) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=0d50ab77397d7278500abd22d34de0e10940b2ee&end=b112bc5529cfa8d8a9000f7a85278ece3232e579&stat=instructions:u)
36+
37+
| (instructions:u) | mean | range | count |
38+
|:----------------------------------:|:----:|:------------:|:-----:|
39+
| Regressions ❌ <br /> (primary) | 0.8% | [0.4%, 1.1%] | 6 |
40+
| Regressions ❌ <br /> (secondary) | - | - | 0 |
41+
| Improvements ✅ <br /> (primary) | - | - | 0 |
42+
| Improvements ✅ <br /> (secondary) | - | - | 0 |
43+
| All ❌✅ (primary) | 0.8% | [0.4%, 1.1%] | 6 |
44+
- Noise as this change only touched test files
45+
46+
47+
Bump version to 1.73 [#113485](https://github.com/rust-lang/rust/pull/113485) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=83964c156db1f444050a38b2498dbd0da6d5d503&end=03247fbbe87b28a991d721c4fe7cdf4e6fc5d5a3&stat=instructions:u)
48+
49+
| (instructions:u) | mean | range | count |
50+
|:----------------------------------:|:----:|:------------:|:-----:|
51+
| Regressions ❌ <br /> (primary) | 0.8% | [0.5%, 1.2%] | 32 |
52+
| Regressions ❌ <br /> (secondary) | 0.7% | [0.7%, 0.7%] | 3 |
53+
| Improvements ✅ <br /> (primary) | - | - | 0 |
54+
| Improvements ✅ <br /> (secondary) | - | - | 0 |
55+
| All ❌✅ (primary) | 0.8% | [0.5%, 1.2%] | 32 |
56+
- Noise as this only was a version bump
57+
58+
59+
Update debuginfo test runner to provide more useful output [#113306](https://github.com/rust-lang/rust/pull/113306) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=1861088f9088344a6b821abf335678d7fe037872&end=ba37a69d30fc6ffc0c2589eb59e936c261b36e76&stat=instructions:u)
60+
61+
| (instructions:u) | mean | range | count |
62+
|:----------------------------------:|:----:|:------------:|:-----:|
63+
| Regressions ❌ <br /> (primary) | - | - | 0 |
64+
| Regressions ❌ <br /> (secondary) | 5.6% | [5.6%, 5.6%] | 1 |
65+
| Improvements ✅ <br /> (primary) | - | - | 0 |
66+
| Improvements ✅ <br /> (secondary) | - | - | 0 |
67+
| All ❌✅ (primary) | - | - | 0 |
68+
- Noise as this change only touched test files
69+
70+
71+
#### Improvements
72+
73+
Move `TyCtxt::mk_x` to `Ty::new_x` where applicable [#113377](https://github.com/rust-lang/rust/pull/113377) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=b112bc5529cfa8d8a9000f7a85278ece3232e579&end=4dd1719b3406d80f539d2f49e9842f3563908632&stat=instructions:u)
74+
75+
| (instructions:u) | mean | range | count |
76+
|:----------------------------------:|:-----:|:--------------:|:-----:|
77+
| Regressions ❌ <br /> (primary) | - | - | 0 |
78+
| Regressions ❌ <br /> (secondary) | - | - | 0 |
79+
| Improvements ✅ <br /> (primary) | -0.7% | [-1.1%, -0.4%] | 10 |
80+
| Improvements ✅ <br /> (secondary) | - | - | 0 |
81+
| All ❌✅ (primary) | -0.7% | [-1.1%, -0.4%] | 10 |
82+
83+
84+
Port PGO/LTO/BOLT optimized build pipeline to Rust [#112235](https://github.com/rust-lang/rust/pull/112235) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=f05d743c204a3c433fb5c2976691fb4eb5c7a530&end=1861088f9088344a6b821abf335678d7fe037872&stat=instructions:u)
85+
86+
| (instructions:u) | mean | range | count |
87+
|:----------------------------------:|:-----:|:--------------:|:-----:|
88+
| Regressions ❌ <br /> (primary) | - | - | 0 |
89+
| Regressions ❌ <br /> (secondary) | 0.3% | [0.3%, 0.3%] | 1 |
90+
| Improvements ✅ <br /> (primary) | -0.9% | [-1.3%, -0.5%] | 38 |
91+
| Improvements ✅ <br /> (secondary) | -0.8% | [-2.3%, -0.3%] | 11 |
92+
| All ❌✅ (primary) | -0.9% | [-1.3%, -0.5%] | 38 |
93+
94+
95+
#### Mixed
96+
97+
perform TokenStream replacement in-place when possible in expand_macro [#113270](https://github.com/rust-lang/rust/pull/113270) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=7cc3da05f99fbc89782fc6cb7e207fa11aa6add5&end=921f669749a57ab5936721fdd93b2da57b581381&stat=instructions:u)
98+
99+
| (instructions:u) | mean | range | count |
100+
|:----------------------------------:|:-----:|:--------------:|:-----:|
101+
| Regressions ❌ <br /> (primary) | 0.6% | [0.6%, 0.7%] | 2 |
102+
| Regressions ❌ <br /> (secondary) | 0.3% | [0.3%, 0.3%] | 3 |
103+
| Improvements ✅ <br /> (primary) | - | - | 0 |
104+
| Improvements ✅ <br /> (secondary) | -0.3% | [-0.3%, -0.3%] | 2 |
105+
| All ❌✅ (primary) | 0.6% | [0.6%, 0.7%] | 2 |
106+
- Might be noise, but otherwise too small to be worth the investigation.
107+
108+
109+
Sync portable-simd to 2023 July 07 [#113437](https://github.com/rust-lang/rust/pull/113437) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=1a449dcfd25143f7e1f6b6f5ddf1c12af361e2ff&end=fd68a6ded951bd7b852ab8107007f7145e3ad6ec&stat=instructions:u)
110+
111+
| (instructions:u) | mean | range | count |
112+
|:----------------------------------:|:-----:|:--------------:|:-----:|
113+
| Regressions ❌ <br /> (primary) | - | - | 0 |
114+
| Regressions ❌ <br /> (secondary) | 1.6% | [1.6%, 1.6%] | 1 |
115+
| Improvements ✅ <br /> (primary) | -1.0% | [-1.5%, -0.6%] | 11 |
116+
| Improvements ✅ <br /> (secondary) | -1.0% | [-1.7%, -0.6%] | 4 |
117+
| All ❌✅ (primary) | -1.0% | [-1.5%, -0.6%] | 11 |
118+
- Numbers were declared acceptable as it potentially could have been much worse.

0 commit comments

Comments
 (0)