Skip to content

Conversation

@atuchin-m
Copy link
Collaborator

@atuchin-m atuchin-m commented Sep 29, 2025

Rust toolchain should be updated by a PR (in a combination with fixing new errors/linter issues)

Tested with 1.89 pinned on a custom branch:
image

@atuchin-m atuchin-m self-assigned this Sep 29, 2025
@atuchin-m atuchin-m requested a review from a team as a code owner September 29, 2025 19:18
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Rust Benchmark

Benchmark suite Current: 07ec4d7 Previous: ea5dc7a Ratio
rule-match-browserlike/brave-list 2263842472 ns/iter (± 12710034) 2254218250 ns/iter (± 9092623) 1.00
rule-match-first-request/brave-list 1056702 ns/iter (± 13627) 1040226 ns/iter (± 7997) 1.02
blocker_new/brave-list 154206140 ns/iter (± 1643597) 155889016 ns/iter (± 929089) 0.99
blocker_new/brave-list-deserialize 67129709 ns/iter (± 1083092) 68301030 ns/iter (± 841217) 0.98
memory-usage/brave-list-initial 18344519 ns/iter (± 3) 18344519 ns/iter (± 3) 1
memory-usage/brave-list-initial/max 66961309 ns/iter (± 3) 66961309 ns/iter (± 3) 1
memory-usage/brave-list-initial/alloc-count 1616130 ns/iter (± 3) 1616130 ns/iter (± 3) 1
memory-usage/brave-list-1000-requests 2551938 ns/iter (± 3) 2551938 ns/iter (± 3) 1
memory-usage/brave-list-1000-requests/alloc-count 68864 ns/iter (± 3) 68864 ns/iter (± 3) 1
url_cosmetic_resources/brave-list 206156 ns/iter (± 2374) 206048 ns/iter (± 1753) 1.00
cosmetic-class-id-match/brave-list 4298312 ns/iter (± 1157888) 4279031 ns/iter (± 1162303) 1.00

This comment was automatically generated by workflow using github-action-benchmark.

@antonok-edm antonok-edm merged commit d380055 into master Sep 30, 2025
9 checks passed
@antonok-edm antonok-edm deleted the pin-rust-toolchain branch September 30, 2025 04:45
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.

4 participants