Merge pull request #1387 from rylev/triage/2022-08-09

Add triage for 2022-08-09

Author: Mark-Simulacrum

triage/2022-08-09.md

@@ -0,0 +1,87 @@
+# 2022-08-09 Triage Log
+
+A pretty quiet week for performance. Unfortunately, by far the biggest change was a regression introduced by increasing the minimum libc version for linux-gnu targets. The exact reason for why this happened in this case is unclear, and it's not easy to investigate. Luckily, the average regression introduced by this change was 0.4% which is fairly small, and many of the larger regressions were limited to doc builds.
+
+Triage done by **@rylev**.
+Revision range: [792bc5a0..cc4dd6fc](https://perf.rust-lang.org/?start=792bc5a0102d0973d42183a2b267850bb905236f&end=cc4dd6fc9f1a5c798df269933c7e442b79661a86&absolute=false&stat=instructions%3Au)
+
+**Summary**:
+
+|            | mean | max | count |
+|:----------:|:----:|:---:|:-----:|
+| Regressions ❌ <br /> (primary) | 0.5% | 1.4% | 146   |
+| Regressions ❌ <br /> (secondary) | 0.8% | 1.6% | 78    |
+| Improvements ✅ <br /> (primary) | N/A  | N/A | 0     |
+| Improvements ✅ <br /> (secondary) | -2.0% | -4.0% | 9     |
+| All ❌✅ (primary) | 0.5% | 1.4% | 146   |
+
+
+1 Regressions, 2 Improvements, 2 Mixed; 1 of them in rollups
+42 artifact comparisons made in total
+
+#### Regressions
+
+Increase the minimum linux-gnu versions [#95026](https://github.com/rust-lang/rust/pull/95026) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=f6f9d5e73d5524b6281c10a5c89b7db35c330634&end=2da8820470817bfd3053be969fb38be9645e1a3d&stat=instructions:u)
+
+|            | mean | max | count |
+|:----------:|:----:|:---:|:-----:|
+| Regressions ❌ <br /> (primary) | 0.4% | 1.6% | 137   |
+| Regressions ❌ <br /> (secondary) | 0.7% | 1.6% | 65    |
+| Improvements ✅ <br /> (primary) | N/A  | N/A | 0     |
+| Improvements ✅ <br /> (secondary) | -0.6% | -0.6% | 2     |
+| All ❌✅ (primary) | 0.4% | 1.6% | 137   |
+- This is difficult to debug as there was no change to code in the project, just in how things are built.
+- Most of the larger regressions are in doc builds where we have slightly higher tolerance for performance issues. 
+- This means the regressions are something we can probably live with.
+
+
+#### Improvements
+
+remove `commit_unconditionally` [#99509](https://github.com/rust-lang/rust/pull/99509) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=e4417cf020fbcd6182c11637bc6b8694434bd81a&end=b759b2efad2d9c713eeab19bf90736a12ff0d74c&stat=instructions:u)
+
+|            | mean | max | count |
+|:----------:|:----:|:---:|:-----:|
+| Regressions ❌ <br /> (primary) | N/A  | N/A | 0     |
+| Regressions ❌ <br /> (secondary) | N/A  | N/A | 0     |
+| Improvements ✅ <br /> (primary) | -0.5% | -1.0% | 8     |
+| Improvements ✅ <br /> (secondary) | -0.8% | -0.8% | 1     |
+| All ❌✅ (primary) | -0.5% | -1.0% | 8     |
+
+
+Rollup of 6 pull requests [#100304](https://github.com/rust-lang/rust/pull/100304) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=8d1fa7105b2ad3154f63d8f0641fd7da87e20e8f&end=5af97e8b0b9649ed14b0b4528a610ff841b6365e&stat=instructions:u)
+
+|            | mean | max | count |
+|:----------:|:----:|:---:|:-----:|
+| Regressions ❌ <br /> (primary) | N/A  | N/A | 0     |
+| Regressions ❌ <br /> (secondary) | 1.5% | 1.5% | 1     |
+| Improvements ✅ <br /> (primary) | N/A  | N/A | 0     |
+| Improvements ✅ <br /> (secondary) | -1.9% | -4.0% | 13    |
+| All ❌✅ (primary) | N/A  | N/A | 0     |
+
+
+#### Mixed
+
+Warn about dead tuple struct fields [#95977](https://github.com/rust-lang/rust/pull/95977) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=cdfd675a63090182fd1c5f2ff58d8eaa115da156&end=9bbbf60b0442f1d56fc39f30274be77acc79164c&stat=instructions:u)
+
+|            | mean | max | count |
+|:----------:|:----:|:---:|:-----:|
+| Regressions ❌ <br /> (primary) | N/A  | N/A | 0     |
+| Regressions ❌ <br /> (secondary) | 1.2% | 1.5% | 3     |
+| Improvements ✅ <br /> (primary) | N/A  | N/A | 0     |
+| Improvements ✅ <br /> (secondary) | -1.4% | -1.6% | 6     |
+| All ❌✅ (primary) | N/A  | N/A | 0     |
+- The regressions are only in secondary tests and are more or less a wash (tipping slightly towards more improvements)
+
+
+Add option to `mir::MutVisitor` to not invalidate CFG. [#100089](https://github.com/rust-lang/rust/pull/100089) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=5af97e8b0b9649ed14b0b4528a610ff841b6365e&end=cc4dd6fc9f1a5c798df269933c7e442b79661a86&stat=instructions:u)
+
+|            | mean | max | count |
+|:----------:|:----:|:---:|:-----:|
+| Regressions ❌ <br /> (primary) | 0.3% | 0.4% | 3     |
+| Regressions ❌ <br /> (secondary) | 1.3% | 1.5% | 6     |
+| Improvements ✅ <br /> (primary) | N/A  | N/A | 0     |
+| Improvements ✅ <br /> (secondary) | -1.5% | -1.5% | 1     |
+| All ❌✅ (primary) | 0.3% | 0.4% | 3     |
+- This is a small enough regression that I don't think we need to investigate it deeply.
+- It seems there's more calls in type unification, but I don't see how this change would impact that. 
+- Left [a comment on the PR](https://github.com/rust-lang/rust/pull/100089#issuecomment-1209611594)