Skip to content

Conversation

@petrochenkov
Copy link

I think this should be enough, if the set of benchmarks to accumulate can be configured by filters (so we could select only incr-patched benchmarks, for example).

Although initially I thought about this being added as one more benchmark in the list (#t-compiler/performance > Average over all benchmarks on the "compare" page @ 💬), but it looks like it's a significantly larger work.

@Kobzol
Copy link
Member Author

Kobzol commented Oct 20, 2025

I think this should be enough, if the set of benchmarks to accumulate can be configured by filters (so we could select only incr-patched benchmarks, for example).

Yes, the accumulated change will automatically update after filters are modified.

Although initially I thought about this being added as one more benchmark in the list

Yeah, that would be more complicated.

@Kobzol Kobzol force-pushed the aggregation-total branch from 7442c8d to ca7d3fc Compare October 20, 2025 12:35
@Kobzol Kobzol force-pushed the aggregation-total branch from ca7d3fc to dd017e8 Compare October 20, 2025 14:36
@Kobzol Kobzol requested a review from Jamesbarford October 21, 2025 07:22
Copy link
Contributor

@Jamesbarford Jamesbarford left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Kobzol Kobzol added this pull request to the merge queue Oct 21, 2025
Merged via the queue into rust-lang:master with commit c8b6fc1 Oct 21, 2025
14 checks passed
@Kobzol Kobzol deleted the aggregation-total branch October 21, 2025 07:39
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.

3 participants