Merge pull request #783 from ecstatic-morse/triage-10-05

Triage 2020-10-05

Author: Mark-Simulacrum

triage/2020-10-05.md

@@ -0,0 +1,47 @@
+# 2020-10-05 Triage Log
+
+A quiet week. One rather large regression on a synthetic benchmark and a few
+small improvements.
+
+[#77023](https://github.com/rust-lang/rust/issues/77023) is an interesting
+case. It encoded an invariant about slice lengths as an `assume` intrinsic
+inside `len` function. It seems to have caused a small compile-time slowdown,
+but there was no improvement in `check` build performance (a proxy for generated
+code quality). In fact, the LLVM documentation [specifically advises
+against](https://llvm.org/docs/LangRef.html#llvm-assume-intrinsic) overuse of
+the `assume` intrinsic in cases where the invariant is unlikely to be of much
+help to the optimizer. That seems to be the case here.
+
+Triage done by **@ecstaticmorse**.
+Revision range: [6369a98ebdee8ce01510f5d4307ddb771c8cb0e5..ea7e131435a960d154e9a5d6a9297039574ffd7d](https://perf.rust-lang.org/?start=6369a98ebdee8ce01510f5d4307ddb771c8cb0e5&end=ea7e131435a960d154e9a5d6a9297039574ffd7d&absolute=false&stat=instructions%3Au)
+
+1 Regressions, 2 Improvements, 1 Mixed
+
+1 of them in rollups
+
+#### Regressions
+
+[#77381](https://github.com/rust-lang/rust/issues/77381) Rollup of 12 pull requests
+- Very large regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=9bb55dc8642d811d66a7599812009cc063577e00&end=b218b952f800c1160b8b5e764ca651b02d678565&stat=instructions:u) (up to 18.3% on `incr-full` builds of `deeply-nested-debug`)
+- I suspect [#76909](https://github.com/rust-lang/rust/pull/76909).
+
+#### Improvements
+
+[#77257](https://github.com/rust-lang/rust/issues/77257) Optimize `IntRange::from_pat`, then shrink `ParamEnv`
+- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=fc2daaae610b5515438b551a2f3706196a997f35&end=48cab6744786cdc5cb5428d2b64efc967ae90496&stat=instructions:u) (up to -2.1% on `full` builds of `ctfe-stress-4-check`)
+- Moderate regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=fc2daaae610b5515438b551a2f3706196a997f35&end=48cab6744786cdc5cb5428d2b64efc967ae90496&stat=instructions:u) (up to 1.3% on `full` builds of `unicode_normalization-check`)
+
+[#77466](https://github.com/rust-lang/rust/issues/77466) Re-land PR #71840 (Rework MIR drop tree lowering) #77466
+- Moderate regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=beb5ae474d2835962ebdf7416bd1c9ad864fe101&end=ced813fec0fb9e883906f18b76d618baf9f5bc08&stat=instructions:u) (up to 2.3% on `full` builds of `regex-opt`)
+- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=beb5ae474d2835962ebdf7416bd1c9ad864fe101&end=ced813fec0fb9e883906f18b76d618baf9f5bc08&stat=instructions:u) (up to -2.2% on `incr-unchanged` builds of `webrender-wrench-opt`)
+
+#### Mixed
+
+[#77023](https://github.com/rust-lang/rust/issues/77023) Hint the maximum length permitted by invariant of slices
+- Moderate regression in [instruction counts](https://perf.rust-lang.org/compare.html?start=4ccf5f731bb71db3470002d6baf5ab4792b821d9&end=beb5ae474d2835962ebdf7416bd1c9ad864fe101&stat=instructions:u) (up to 2.5% on `incr-patched: sparse set` builds of `regex-opt`)
+- Moderate improvement in [instruction counts](https://perf.rust-lang.org/compare.html?start=4ccf5f731bb71db3470002d6baf5ab4792b821d9&end=beb5ae474d2835962ebdf7416bd1c9ad864fe101&stat=instructions:u) (up to -1.2% on `incr-patched: b9b3e592dd cherry picked` builds of `style-servo-debug`)
+
+#### Nags requiring follow up
+
+- [#77381](https://github.com/rust-lang/rust/issues/77381)
+- [#77023](https://github.com/rust-lang/rust/issues/77023)