Skip to content

Add perf triage for 2026-07-06#2492

Merged
panstromek merged 2 commits into
masterfrom
panstromek-patch-1
Jul 7, 2026
Merged

Add perf triage for 2026-07-06#2492
panstromek merged 2 commits into
masterfrom
panstromek-patch-1

Conversation

@panstromek

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread triage/2026/2026-07-06.md
Comment on lines +42 to +43
Use `LowerAvailableDepth::No` for normalizes-to goal itself instead of its nested goals [#158705](https://github.com/rust-lang/rust/pull/158705) [(Comparison Link)](https://perf.rust-lang.org/compare.html?start=3659db0d3e2cd634c766fcda79ed118eca31a9fd&end=3c00c96d3af4d5b5e101e56cc161a608b21366ee&stat=instructions:u)

@panstromek panstromek Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's something weird going on. Base commit of this PR is the end commit of the summary range. It looks like summary range "forgets" the last commit during generation. Even if I specify it explicitly in the url, it will silently only include the one before.

I think this is the same issue as #1599. I think that that bug will happen if the last commit in range has significant changes, so next triage will reproduce it, I believe.

I suspect we do something suspicious when iterating over commits, but I tried to stare at the main loop in handle_triage for an hour and I couldn't figure out whether it's right or not, it is quite confusing 😄

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opened an issue for this #2493

Comment thread triage/2026/2026-07-06.md Outdated
Co-authored-by: Nicholas Nethercote <n.nethercote@gmail.com>
@panstromek panstromek enabled auto-merge July 7, 2026 05:22
@panstromek panstromek added this pull request to the merge queue Jul 7, 2026
Merged via the queue into master with commit 6935412 Jul 7, 2026
14 checks passed
@panstromek panstromek deleted the panstromek-patch-1 branch July 7, 2026 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants