Skip to content

sort: remove some noise from benchmark#11237

Closed
oech3 wants to merge 3 commits intouutils:mainfrom
oech3:sort-bench-noise
Closed

sort: remove some noise from benchmark#11237
oech3 wants to merge 3 commits intouutils:mainfrom
oech3:sort-bench-noise

Conversation

@oech3
Copy link
Contributor

@oech3 oech3 commented Mar 8, 2026

No description provided.

@oech3 oech3 force-pushed the sort-bench-noise branch 2 times, most recently from 281b4fe to 928190e Compare March 8, 2026 12:43
@xtqqczze
Copy link
Contributor

xtqqczze commented Mar 8, 2026

this is the same thing as #10388

@oech3
Copy link
Contributor Author

oech3 commented Mar 8, 2026

yes. But I hurry this for sort and large 1 PR is difficult to review and merge.
Please split it. Then I close this.

@oech3 oech3 closed this Mar 8, 2026
@oech3 oech3 deleted the sort-bench-noise branch March 8, 2026 12:51
@xtqqczze
Copy link
Contributor

xtqqczze commented Mar 8, 2026

yeah, I didn't see any improvement for sort benchmarks

@oech3
Copy link
Contributor Author

oech3 commented Mar 8, 2026

It might have different result for custom allocator.

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