Skip to content

Commit

Permalink
Merge pull request #2033 from Mark-Simulacrum/triage
Browse files Browse the repository at this point in the history
Add triage for this week
  • Loading branch information
Kobzol authored Jan 20, 2025
2 parents 3b209bb + ac87015 commit c4ee79c
Showing 1 changed file with 67 additions and 0 deletions.
67 changes: 67 additions & 0 deletions triage/2025-01-20.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,67 @@
# 2025-01-20 Triage Log

A very quiet week for performance, with small improvements essentially on all
benchmarks.

Triage done by **@simulacrum**.
Revision range: [1ab85fbd..9a1d156f](https://perf.rust-lang.org/?start=1ab85fbd7474e8ce84d5283548f21472860de3e2&end=9a1d156f38c51441ee51e5a068f1d0caf4bb0f27&absolute=false&stat=instructions%3Au)

**Summary**:

| (instructions:u) | mean | range | count |
|:----------------------------------:|:-----:|:--------------:|:-----:|
| Regressions ❌ <br /> (primary) | 0.7% | [0.3%, 1.0%] | 3 |
| Regressions ❌ <br /> (secondary) | 0.7% | [0.5%, 1.0%] | 4 |
| Improvements ✅ <br /> (primary) | -0.4% | [-1.3%, -0.1%] | 155 |
| Improvements ✅ <br /> (secondary) | -0.5% | [-1.8%, -0.1%] | 94 |
| All ❌✅ (primary) | -0.4% | [-1.3%, 1.0%] | 158 |


0 Regression, 1 Improvement, 2 Mixed; 0 of them in rollups
40 artifact comparisons made in total

#### Regressions

No statistically significant regressions this cycle.

#### Improvements

Stable Hash: Ignore all HirIds that just identify the node itself [#135329](https://github.com/rust-lang/rust/pull/135329) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=bcd0683e5dce1945b5d940714742e7502883bb5c&end=6067b36314ab5eb2eb47cecc464545ba58e1ad24&stat=instructions:u)

| (instructions:u) | mean | range | count |
|:----------------------------------:|:-----:|:--------------:|:-----:|
| Regressions ❌ <br /> (primary) | - | - | 0 |
| Regressions ❌ <br /> (secondary) | - | - | 0 |
| Improvements ✅ <br /> (primary) | -0.4% | [-0.9%, -0.1%] | 169 |
| Improvements ✅ <br /> (secondary) | -0.5% | [-1.3%, -0.1%] | 85 |
| All ❌✅ (primary) | -0.4% | [-0.9%, -0.1%] | 169 |


#### Mixed

Less unsafe in `dangling`/`without_provenance` [#135344](https://github.com/rust-lang/rust/pull/135344) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=d61f55d8b9d4703207a5980f27b6c28973ba27ee&end=d8a64098c9d0fb25699f657c6efff0bb418f7e18&stat=instructions:u)

| (instructions:u) | mean | range | count |
|:----------------------------------:|:-----:|:--------------:|:-----:|
| Regressions ❌ <br /> (primary) | 1.2% | [0.4%, 2.1%] | 2 |
| Regressions ❌ <br /> (secondary) | - | - | 0 |
| Improvements ✅ <br /> (primary) | -0.5% | [-0.9%, -0.3%] | 3 |
| Improvements ✅ <br /> (secondary) | -1.1% | [-1.6%, -0.6%] | 4 |
| All ❌✅ (primary) | 0.2% | [-0.9%, 2.1%] | 5 |

Regressions look likely to be noise/modality in benchmarks, not genuine changes
caused by this work. See also some
[analysis](https://github.com/rust-lang/rust/pull/135344#issuecomment-2591403244)
on the PR.

Temporarily bring back `Rvalue::Len` [#135709](https://github.com/rust-lang/rust/pull/135709) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=98572840b6da350e5615a68586a766a0f9f1470a&end=c62b732724b6beb50a5c73be26d0c5f668504059&stat=instructions:u)

| (instructions:u) | mean | range | count |
|:----------------------------------:|:-----:|:--------------:|:-----:|
| Regressions ❌ <br /> (primary) | 0.3% | [0.3%, 0.3%] | 1 |
| Regressions ❌ <br /> (secondary) | 0.4% | [0.2%, 0.5%] | 6 |
| Improvements ✅ <br /> (primary) | -0.7% | [-1.3%, -0.4%] | 3 |
| Improvements ✅ <br /> (secondary) | -0.9% | [-1.7%, -0.2%] | 2 |
| All ❌✅ (primary) | -0.4% | [-1.3%, 0.3%] | 4 |

Fixing P-critical regression, so easily justified.

0 comments on commit c4ee79c

Please sign in to comment.