Skip to content

Commit de54126

Browse files
rylevrylev
authored
Merge pull request #2005 from rylev/perf-triage-2024-11-05
Add perf triage for 2024-11-05
2 parents fdcfe88 + b232a70 commit de54126

File tree

1 file changed

+111
-0
lines changed

1 file changed

+111
-0
lines changed

‎triage/2024-11-05.md

Lines changed: 111 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,111 @@
1+
# 2024-11-05 Triage Log
2+
3+
A week dominated by one large improvement and one large regression where luckily the improvement had a larger impact. The regression seems to have been caused by a newly introduced lint that might have performance issues. The improvement was in building rustc with protected visibility which reduces the number of dynamic relocations needed leading to some nice performance gains. Across a large swath of the perf suite, the compiler is on average 1% faster after this week compared to last week.
4+
5+
Triage done by **@rylev**.
6+
Revision range: [c8a8c820..27e38f8f](https://perf.rust-lang.org/?start=c8a8c82035439cb2404b8f24ca0bc18209d534ca&end=27e38f8fc7efc57b75e9a763d7a0ee44822cd5f7&absolute=false&stat=instructions%3Au)
7+
8+
**Summary**:
9+
10+
| (instructions:u) | mean | range | count |
11+
|:----------------------------------:|:-----:|:---------------:|:-----:|
12+
| Regressions ❌ <br /> (primary) | 0.8% | [0.1%, 2.0%] | 80 |
13+
| Regressions ❌ <br /> (secondary) | 1.9% | [0.2%, 3.4%] | 45 |
14+
| Improvements ✅ <br /> (primary) | -1.9% | [-31.6%, -0.1%] | 148 |
15+
| Improvements ✅ <br /> (secondary) | -5.1% | [-27.8%, -0.1%] | 180 |
16+
| All ❌✅ (primary) | -1.0% | [-31.6%, 2.0%] | 228 |
17+
18+
19+
1 Regression, 1 Improvement, 5 Mixed; 3 of them in rollups
20+
46 artifact comparisons made in total
21+
22+
#### Regressions
23+
24+
Rollup of 6 pull requests [#132326](https://github.com/rust-lang/rust/pull/132326) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=e473783d90e2289b8a97575fa60d6315f0a318eb&end=1e4f10ba6476e48a42a79b9f846a2d9366525b9e&stat=instructions:u)
25+
26+
| (instructions:u) | mean | range | count |
27+
|:----------------------------------:|:----:|:------------:|:-----:|
28+
| Regressions ❌ <br /> (primary) | 0.7% | [0.2%, 1.9%] | 104 |
29+
| Regressions ❌ <br /> (secondary) | 1.5% | [0.1%, 4.4%] | 68 |
30+
| Improvements ✅ <br /> (primary) | - | - | 0 |
31+
| Improvements ✅ <br /> (secondary) | - | - | 0 |
32+
| All ❌✅ (primary) | 0.7% | [0.2%, 1.9%] | 104 |
33+
- Fairly large regression caused by https://github.com/rust-lang/rust/pull/131984.
34+
- Most likely due to a non-performant lint being run that wasn't previously.
35+
- Checked with the author and reviewer.
36+
37+
38+
#### Improvements
39+
40+
Use protected visibility when building rustc with LLD [#131634](https://github.com/rust-lang/rust/pull/131634) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=a8e1186e3c14a54f7a38cc1183117dc7e77f4f82&end=9fa9ef385c0aad8f5d4c8f7d92dca474367943a3&stat=instructions:u)
41+
42+
| (instructions:u) | mean | range | count |
43+
|:----------------------------------:|:-----:|:---------------:|:-----:|
44+
| Regressions ❌ <br /> (primary) | - | - | 0 |
45+
| Regressions ❌ <br /> (secondary) | - | - | 0 |
46+
| Improvements ✅ <br /> (primary) | -1.8% | [-31.3%, -0.1%] | 174 |
47+
| Improvements ✅ <br /> (secondary) | -4.2% | [-27.5%, -0.2%] | 224 |
48+
| All ❌✅ (primary) | -1.8% | [-31.3%, -0.1%] | 174 |
49+
50+
51+
#### Mixed
52+
53+
Rollup of 12 pull requests [#132317](https://github.com/rust-lang/rust/pull/132317) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=c8a8c82035439cb2404b8f24ca0bc18209d534ca&end=2dece5bb62f234f5622a08289c5a3d1555cd7843&stat=instructions:u)
54+
55+
| (instructions:u) | mean | range | count |
56+
|:----------------------------------:|:-----:|:--------------:|:-----:|
57+
| Regressions ❌ <br /> (primary) | 0.2% | [0.2%, 0.4%] | 4 |
58+
| Regressions ❌ <br /> (secondary) | 0.2% | [0.2%, 0.2%] | 1 |
59+
| Improvements ✅ <br /> (primary) | -0.2% | [-0.4%, -0.1%] | 6 |
60+
| Improvements ✅ <br /> (secondary) | -0.3% | [-0.3%, -0.3%] | 1 |
61+
| All ❌✅ (primary) | -0.0% | [-0.4%, 0.4%] | 10 |
62+
- Looks like a bit more time spent in LLVM? Might have been caused by a few PRs in this rollup, but the total result is a wash, so I don't think it's needed to dig deeper into this.
63+
64+
65+
Rc/Arc: don't leak the allocation if drop panics [#132231](https://github.com/rust-lang/rust/pull/132231) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=2dece5bb62f234f5622a08289c5a3d1555cd7843&end=e473783d90e2289b8a97575fa60d6315f0a318eb&stat=instructions:u)
66+
67+
| (instructions:u) | mean | range | count |
68+
|:----------------------------------:|:-----:|:--------------:|:-----:|
69+
| Regressions ❌ <br /> (primary) | 0.6% | [0.6%, 0.6%] | 1 |
70+
| Regressions ❌ <br /> (secondary) | 2.3% | [1.1%, 3.5%] | 6 |
71+
| Improvements ✅ <br /> (primary) | -0.9% | [-1.4%, -0.3%] | 3 |
72+
| Improvements ✅ <br /> (secondary) | -0.4% | [-0.6%, -0.2%] | 5 |
73+
| All ❌✅ (primary) | -0.5% | [-1.4%, 0.6%] | 4 |
74+
- There probably isn't too much to be done regarding perf, all of the major regressions come from one small stress test crate.
75+
76+
77+
Mark `simplify_aggregate_to_copy` mir-opt as unsound [#132356](https://github.com/rust-lang/rust/pull/132356) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=20c909ff9cdd88d33768a4ddb8952927a675b0ad&end=a0d98ff0e5b6e1f2c63fd26f68484792621b235c&stat=instructions:u)
78+
79+
| (instructions:u) | mean | range | count |
80+
|:----------------------------------:|:-----:|:--------------:|:-----:|
81+
| Regressions ❌ <br /> (primary) | 0.7% | [0.3%, 1.9%] | 14 |
82+
| Regressions ❌ <br /> (secondary) | 0.7% | [0.2%, 2.3%] | 16 |
83+
| Improvements ✅ <br /> (primary) | -0.3% | [-0.3%, -0.3%] | 1 |
84+
| Improvements ✅ <br /> (secondary) | -0.3% | [-0.4%, -0.3%] | 5 |
85+
| All ❌✅ (primary) | 0.7% | [-0.3%, 1.9%] | 15 |
86+
- This deactivated an unsound MIR optimization until it is fixed.
87+
88+
89+
add const_panic macro to make it easier to fall back to non-formatting panic in const [#132542](https://github.com/rust-lang/rust/pull/132542) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=7028d9318fadc20e5e3058d52e44d785d31a6aaa&end=e3a918ece026cec748fc64af5b4983095b46097e&stat=instructions:u)
90+
91+
| (instructions:u) | mean | range | count |
92+
|:----------------------------------:|:-----:|:--------------:|:-----:|
93+
| Regressions ❌ <br /> (primary) | 0.8% | [0.2%, 1.8%] | 6 |
94+
| Regressions ❌ <br /> (secondary) | - | - | 0 |
95+
| Improvements ✅ <br /> (primary) | -0.5% | [-0.9%, -0.1%] | 5 |
96+
| Improvements ✅ <br /> (secondary) | - | - | 0 |
97+
| All ❌✅ (primary) | 0.2% | [-0.9%, 1.8%] | 11 |
98+
- Looks like most of the regressions are taking longer in LLVM related queries (e.g., LLVM_module_codegen_emit_obj) which doesn't make a ton of sense to me.
99+
100+
101+
Rollup of 11 pull requests [#132626](https://github.com/rust-lang/rust/pull/132626) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=96477c55bcfa7121470ed225f602c1c639aeaa18&end=27e38f8fc7efc57b75e9a763d7a0ee44822cd5f7&stat=instructions:u)
102+
103+
| (instructions:u) | mean | range | count |
104+
|:----------------------------------:|:-----:|:--------------:|:-----:|
105+
| Regressions ❌ <br /> (primary) | - | - | 0 |
106+
| Regressions ❌ <br /> (secondary) | 0.6% | [0.5%, 0.9%] | 7 |
107+
| Improvements ✅ <br /> (primary) | - | - | 0 |
108+
| Improvements ✅ <br /> (secondary) | -0.3% | [-0.3%, -0.3%] | 2 |
109+
| All ❌✅ (primary) | - | - | 0 |
110+
- It's not obvious to me which PR might be the culprit here and the regression is small and contained enough that I don't think it's worth investigating.
111+

0 commit comments

Comments
 (0)