Skip to content

✨ countZeroBytes for bytes#1414

Merged
Vectorized merged 1 commit into
mainfrom
count-zero-bytes-memory
May 7, 2025
Merged

✨ countZeroBytes for bytes#1414
Vectorized merged 1 commit into
mainfrom
count-zero-bytes-memory

Conversation

@Vectorized
Copy link
Copy Markdown
Owner

@Vectorized Vectorized commented May 7, 2025

Description

Maybe this should be in LibBytes?

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
Copy Markdown

github-actions Bot commented May 7, 2025

Gas Snapshot Comparison Report

Generated at commit : 5c06b44, Compared to commit : b423060

Contract Name Test Name Main Gas PR Gas Diff
LibBitTest testAnd() 192766 192767 1
testCLZ() 306426 306404 -22
testCommonBytePrefix() 1555 1533 -22
testIsPo2() 57707 57685 -22
testPassInBool() 829 896 67
testPopCount() 66842 66798 -44
testReverseBytes() 12536 12537 1
test__codesize() 7124 8202 1078

@Vectorized Vectorized merged commit c19ac1f into main May 7, 2025
13 checks passed
@Vectorized Vectorized deleted the count-zero-bytes-memory branch May 7, 2025 18:46
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.

1 participant