Skip to content

Commit 5a1727e

Browse files
authored
Add triage report for 2020-11-10 (#793)
1 parent 4c136f0 commit 5a1727e

File tree

1 file changed

+39
-0
lines changed

1 file changed

+39
-0
lines changed

triage/2020-11-10.md

Lines changed: 39 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
1+
# 2020-11-10 Triage Log
2+
3+
A mixed week with improvements still outweighing regressions. Perhaps the biggest highlight was the move to compiling rustc crates [with the initial-exec TLS model](https://github.com/rust-lang/rust/pull/78201) which results in fewer calls to `_tls_get_addr` and thus faster compile times.
4+
5+
Triage done by **@rylevick**.
6+
Revision range: [5cdf5b882da9e8b7c73b5cadeb7745cb68f6ff63..cf9cf7c923eb01146971429044f216a3ca905e06](https://perf.rust-lang.org/?start=5cdf5b882da9e8b7c73b5cadeb7745cb68f6ff63&end=cf9cf7c923eb01146971429044f216a3ca905e06&absolute=false&stat=instructions%3Au)
7+
8+
1 Regressions, 2 Improvements, 2 Mixed
9+
10+
#### Regressions
11+
12+
[#78267](https://github.com/rust-lang/rust/issues/78267)
13+
- Small regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=f92b931045dabb00b892519d3451cb41d41f2d31&end=8532e742fc6ec210fab69b8192190bc40c685912&stat=instructions:u) (up to 1.2% on `full` builds of `deeply-nested-async-check`)
14+
- This might be noise as this only affects one benchmark negatively, and that benchmark tends to be on the noisier side.
15+
#### Improvements
16+
17+
[#78280](https://github.com/rust-lang/rust/issues/78280)
18+
- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=75f1db1102076e416e1154b241b4fc95c01c0d7b&end=89631663b7ad2d46d3e4f52bcfa7bee2be9eb82b&stat=instructions:u) (up to -1.3% on `incr-patched: new row` builds of `tuple-stress-opt`)
19+
20+
[#78201](https://github.com/rust-lang/rust/issues/78201)
21+
- Large improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=fe8f02690804d5ee696bd3bca9515f5f71857e3b&end=25f6938da459a57b43bdf16ed6bdad3225b2a3ce&stat=instructions:u) (up to -7.2% on `incr-full` builds of `webrender-wrench-check`)
22+
- This change may produce similar performance gains in related tooling such as rustdoc and clippy.
23+
24+
#### Mixed
25+
26+
[#77227](https://github.com/rust-lang/rust/issues/77227)
27+
- Large improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=601c13c6fda6a7db423c974797e36c79a9a0c0ac&end=75f1db1102076e416e1154b241b4fc95c01c0d7b&stat=instructions:u) (up to -5.0% on `incr-unchanged` builds of `deeply-nested-async-check`)
28+
- Moderate regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=601c13c6fda6a7db423c974797e36c79a9a0c0ac&end=75f1db1102076e416e1154b241b4fc95c01c0d7b&stat=instructions:u) (up to 1.3% on `full` builds of `ctfe-stress-4-check`)
29+
30+
[#78410](https://github.com/rust-lang/rust/issues/78410)
31+
- Very large improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=f2ea2f648e117013b0217f001088ae89e0f163ca&end=87a0997ef9c0bfad0ba362741afa601d8fb285b8&stat=instructions:u) (up to -26.0% on `incr-unchanged` builds of `deeply-nested-async-opt`)
32+
- Moderate regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=f2ea2f648e117013b0217f001088ae89e0f163ca&end=87a0997ef9c0bfad0ba362741afa601d8fb285b8&stat=instructions:u) (up to 3.8% on `full` builds of `ctfe-stress-4-check`)
33+
- This change is a revert of [a previous change](https://github.com/rust-lang/rust/pull/75443), and at least one user was reporting [massive performance gains](https://github.com/rust-lang/rust/pull/78410#issuecomment-716829861).
34+
35+
#### Nags requiring follow up
36+
37+
The compiler team is once again requested to look into:
38+
39+
- <https://github.com/rust-lang/rust/pull/78432#issuecomment-721388323>

0 commit comments

Comments
 (0)