Skip to content

Conversation

@Vectorized
Copy link
Owner

Description

No new memory allocations. O(n log n) average case using a very simple quicksort.

Checklist

Ensure you completed all of the steps below before submitting your pull request:

  • Ran forge fmt?
  • Ran forge test?

Pull requests with an incomplete checklist will be thrown out.

@github-actions
Copy link

Gas Snapshot Comparison Report

Generated at commit : d6e8ee4, Compared to commit : f281ae5

Contract Name Test Name Main Gas PR Gas Diff
LibSortTest testGroupSum() 35995 31547 -4448
test__codesize() 25695 25804 109

@Vectorized Vectorized merged commit 0c19df1 into main Jun 21, 2025
13 checks passed
@Vectorized Vectorized deleted the opt-group-sum branch June 21, 2025 03:14
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