|
| 1 | +# 2021-03-10 Triage Log |
| 2 | + |
| 3 | +A generally positive albeit quiet week though many of the perf improvements were gaining performance back from previous regressions. We'll need to continue to keep an eye on rollups as there were two that caused small performance changes. |
| 4 | + |
| 5 | +Triage done by **@rylev**. |
| 6 | +Revision range: [edeee915b1c52f97411e57ef6b1a8bd46548a37a..861872bc453bde79b83ff99d443d035225f10e87](https://perf.rust-lang.org/?start=edeee915b1c52f97411e57ef6b1a8bd46548a37a&end=861872bc453bde79b83ff99d443d035225f10e87&absolute=false&stat=instructions%3Au) |
| 7 | + |
| 8 | +1 Regression, 4 Improvements, 1 Mixed |
| 9 | +2 of them in rollups |
| 10 | + |
| 11 | +#### Regressions |
| 12 | + |
| 13 | +Rollup of 8 pull requests [#82929](https://github.com/rust-lang/rust/issues/82929) |
| 14 | +- Moderate regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=4b9f5cc4c10a161047475cb9bbe02c4fda57fb07&end=3a5d45f68cadc8fff4fbb557780f92b403b19c19&stat=instructions:u) (up to 1.6% on `full` builds of `keccak-debug`) |
| 15 | +- A fairly consistent (albeit small) regression in the `typecheck` query across different benchmarks though none of the PRs in the rollup seem to touch that query. This might be noise, but it's hard to tell |
| 16 | + |
| 17 | +#### Improvements |
| 18 | + |
| 19 | +Move check only relevant in error case out of critical path [#82738](https://github.com/rust-lang/rust/issues/82738) |
| 20 | +- Large improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=51748a8fc77283914d4135f31b5966a407208187&end=dfe519b344b60e54a2028d3366ca366a228537e3&stat=instructions:u) (up to -5.3% on `incr-patched: println` builds of `unicode_normalization-check`) |
| 21 | +- Fixes the performance regression introduced in [#81458](https://github.com/rust-lang/rust/pull/81458). |
| 22 | + |
| 23 | +Backport some LLVM compile-time improvements [#82783](https://github.com/rust-lang/rust/issues/82783) |
| 24 | +- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=27885a94c6a6c98a0d49af3962f1218adcd6efd2&end=1d6b0f626aad4ee9f2eaec4d5582f45620ccab80&stat=instructions:u) (up to -2.4% on `incr-patched: println` builds of `regex-opt`) |
| 25 | +- This gains some of the perf lost by the update to LLVM 12 in [#81451](https://github.com/rust-lang/rust/issues/81451). |
| 26 | + |
| 27 | +Rollup of 10 pull requests [#82953](https://github.com/rust-lang/rust/issues/82953) |
| 28 | +- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=3a5d45f68cadc8fff4fbb557780f92b403b19c19&end=861872bc453bde79b83ff99d443d035225f10e87&stat=instructions:u) (up to -2.2% on `incr-patched: b9b3e592dd cherry picked` builds of `style-servo-debug`) |
| 29 | + |
| 30 | +Update tracing to 0.1.25 [#82553](https://github.com/rust-lang/rust/issues/82553) |
| 31 | +- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=939b14334dfec68d85b01b62c1be0172cee03339&end=476acbf1e9965b5e95c90f0d7d658709812b7003&stat=instructions:u) (up to -3.7% on `full` builds of `deeply-nested-check`) |
| 32 | +- Small improvements likely due to slight reduction in code generated for statically disabled instrumentation. |
| 33 | + |
| 34 | +#### Mixed |
| 35 | + |
| 36 | +Upgrade to LLVM 12 [#81451](https://github.com/rust-lang/rust/issues/81451) |
| 37 | +- Very large improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=ec7f258d543e1ac7d0b94435972331e85da8c509&end=409920873cf8a95739a55dc5fe5adb05e1b4758e&stat=instructions:u) (up to -11.3% on `full` builds of `keccak-opt`) |
| 38 | +- Large regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=ec7f258d543e1ac7d0b94435972331e85da8c509&end=409920873cf8a95739a55dc5fe5adb05e1b4758e&stat=instructions:u) (up to 7.7% on `full` builds of `deeply-nested-opt`) |
| 39 | +- Some of the perf regression seen here was gained back in [#82783](https://github.com/rust-lang/rust/issues/82783). |
| 40 | + |
| 41 | +#### Nags requiring follow up |
| 42 | + |
| 43 | +No nags for this week and the nag from last week has been resolved by [#81458](https://github.com/rust-lang/rust/pull/81458). |
0 commit comments