Skip to content

Commit c0af979

Browse files
authored
Merge pull request #1539 from ecstatic-morse/patch-6
Perf triage this week
2 parents 92de987 + 317c306 commit c0af979

File tree

1 file changed

+10
-12
lines changed

1 file changed

+10
-12
lines changed

draft/2020-09-30-this-week-in-rust.md

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -96,21 +96,19 @@ If you are a Rust project owner and are looking for contributors, please submit
9696

9797
## Rust Compiler Performance Triage
9898

99-
* [2020-09-21](https://github.com/rust-lang/rustc-perf/blob/master/triage/2020-09-21.md):
100-
2 Regressions, 5 Improvements, 4 Mixed
99+
* [2020-09-28](https://github.com/rust-lang/rustc-perf/blob/master/triage/2020-09-28.md):
100+
0 Regressions, 1 Improvements, 3 Mixed
101101

102-
This was the first week of semi-automated perf triage, and thank goodness:
103-
There was a lot going on. Most regressions are either quite small or already
104-
have a fix published.
105102

106-
[#72412](https://github.com/rust-lang/rust/issues/72412) is probably the most
107-
interesting case. It fixes a pathological problem involving nested closures by
108-
adding cycle detection to what seems to be a relatively hot part of the code.
109-
As a result, most users will see a slight [compile-time
110-
regression](https://perf.rust-lang.org/compare.html?start=2c69266c0697b0c0b34abea62cba1a1d3c59c90c&end=fdc3405c20122fd0f077f5a77addabc873f20e4c&stat=task-clock)
111-
for their crates.
103+
Most significant changes this week came in response to regressions discussed in
104+
last week's triage report. Curious readers may be interested in
105+
[#77058](https://github.com/rust-lang/rust/issues/77058), in which the removal
106+
of a single field from a struct caused a 25% decrease in wall-times for one
107+
seemingly unrelated benchmark, or
108+
[#76986](https://github.com/rust-lang/rust/issues/76986), an ABI change that
109+
should be a pretty clear win but seems to have mixed results.
112110

113-
See the [full report](https://github.com/rust-lang/rustc-perf/blob/master/triage/2020-09-21.md) for more.
111+
See the [full report](https://github.com/rust-lang/rustc-perf/blob/master/triage/2020-09-28.md) for more.
114112

115113
## Approved RFCs
116114

0 commit comments

Comments
 (0)