Skip to content

Conversation

@Vectorized
Copy link
Owner

@Vectorized Vectorized commented Apr 22, 2025

Description

Faster better stronger.

Tbh, we really need a clz opcode.

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

github-actions bot commented Apr 22, 2025

Gas Snapshot Comparison Report

Generated at commit : 8648b21, Compared to commit : d76362f

Contract Name Test Name Main Gas PR Gas Diff
ERC4337Test testCdFallback2() 1069638 1067596 -2042
test__codesize() 56042 56049 7
ERC6551Test testCdFallback() 825856 823785 -2071
test__codesize() 51596 51603 7
LibZipTest testCdCompress() 26367 23866 -2501
testCdCompressGas() 8262 7665 -597
testCdDecompressOnInvalidInput() 1353 1294 -59
testCdFallback() 2214309 2151515 -62794
testFlzCompressDecompress() 1245097 1249322 4225
testFlzCompressDecompress2() 754846 757869 3023
testStoreABCGas() 66540 66496 -44
testStoreABCPackedGas() 22285 22308 23
testStoreABCWithCdCompressGas() 25844 - -
testStoreABCWithCdCompressOriginalGas() 36188 - -
testStoreABCWithFlzCompressGas() 68488 - -
test__codesize() 21945 23961 2016
testABCCdCompressAndDecompressGas() - 4806 -
testABCCdCompressAndDecompressOriginalGas() - 16279 -
testABCStoreWithCdCompressGas() - 25455 -
testABCStoreWithCdCompressOriginalGas() - 36232 -
testABCStoreWithFlzCompressGas() - 68465 -
testCdDecompressGas() - 2722 -
testCdDecompressOriginalGas() - 6178 -

@Vectorized Vectorized merged commit 325d888 into main Apr 22, 2025
13 checks passed
@Vectorized Vectorized deleted the opt-cdcompress-more branch April 22, 2025 13:20
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