Skip to content

Commit a233e23

Browse files
finish report
1 parent 03a84b8 commit a233e23

File tree

1 file changed

+19
-10
lines changed

1 file changed

+19
-10
lines changed

triage/2021-01-26.md

Lines changed: 19 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,27 @@
1-
2021-01-26 Triage Log
1+
# 2021-01-26 Triage Log
22

3-
TODO: Summary
3+
This week continues a trend of relatively large rollups, which often contain
4+
perf-sensitive PRs. We need to get better at marking PRs as rollup=never or
5+
otherwise not including them, but it is unclear how precisely to go about doing
6+
so. The tooling for testing individual PRs merged in rollups should also be
7+
improved to work better in the next few days, though.
48

5-
Triage done by **@rylevick**.
9+
Other than that, this week saw several regressions, most of which were not
10+
easily explained. We are seeking feedback from PR authors and reviewers on
11+
whether the results are expected and if anything can be done.
12+
13+
Triage done by **@rylevick** and **@simulacrum**.
614
Revision range: [e05409a02c6e73a3dea6da98798468db2910ca59..1483e67addd37d9bd20ba3b4613b678ee9ad4d68](https://perf.rust-lang.org/?start=e05409a02c6e73a3dea6da98798468db2910ca59&end=1483e67addd37d9bd20ba3b4613b678ee9ad4d68&absolute=false&stat=instructions%3Au)
715

8-
5 Regressions, 2 Improvements, 2 Mixed
9-
??? of them in rollups
16+
5 Regressions, 2 Improvements, 1 Mixed;
17+
3 of them in rollups
1018

1119
#### Regressions
1220

1321
Deprecate-in-future the constants superceded by RFC 2700 [#80958](https://github.com/rust-lang/rust/issues/80958)
1422
- Small regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=57a71ac0e17e4f7070b090ab7bdc5474d8e37ecc&end=339e19697a39a78f4d669c217b7d21109215de41&stat=instructions:u) (up to 2.4% on `incr-patched: Job` builds of `regex-check`)
15-
- Unsure why this might be causing performance regressions as this change does not touch much code and only produces lint warnings when the user opts into them.
16-
- The author of the PR [was asked why this might be the case](https://github.com/rust-lang/rust/pull/80958#issuecomment-767837811).
23+
- Unsure why this might be causing performance regressions as this change does not touch much code and only produces lint warnings when the user opts into them.
24+
- The author of the PR [was asked why this might be the case](https://github.com/rust-lang/rust/pull/80958#issuecomment-767837811), and a nag noted.
1725

1826
Rollup of 11 pull requests [#81240](https://github.com/rust-lang/rust/issues/81240)
1927
- Moderate regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=339e19697a39a78f4d669c217b7d21109215de41&end=a243ad280a4ac57c1b8427e30e31c5b505cf10de&stat=instructions:u) (up to 2.0% on `full` builds of `keccak-check`)
@@ -29,6 +37,7 @@ Rollup of 14 pull requests [#81355](https://github.com/rust-lang/rust/issues/813
2937
Prevent query cycles in the MIR inliner [#68828](https://github.com/rust-lang/rust/issues/68828)
3038
- Moderate regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=7fba12bb1d3877870758a7a53e2fe766bb19bd60&end=f4eb5d9f719cd3c849befc8914ad8ce0ddcf34ed&stat=instructions:u) (up to 2.1% on `full` builds of `keccak-check`)
3139
- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=7fba12bb1d3877870758a7a53e2fe766bb19bd60&end=f4eb5d9f719cd3c849befc8914ad8ce0ddcf34ed&stat=instructions:u) (up to -1.8% on `full` builds of `ctfe-stress-4-check`)
40+
- Nag filed
3241

3342
#### Improvements
3443

@@ -46,6 +55,6 @@ mark raw_vec::ptr with inline [#79113](https://github.com/rust-lang/rust/issues/
4655

4756
#### Nags requiring follow up
4857

49-
TODO: Nags
50-
51-
58+
- Deprecate in future regression [#80958](https://github.com/rust-lang/rust/pull/80958#issuecomment-767837811)
59+
- Waiting on [response](https://github.com/rust-lang/rust/pull/80919#issuecomment-767854932) to inquiry.
60+
- Regression in prevent query cycles in the MIR inliner - expected? [#68828](https://github.com/rust-lang/rust/pull/68828#issuecomment-767872361)

0 commit comments

Comments
 (0)