Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨ Coalesce #1360

Merged
merged 3 commits into from
Feb 20, 2025
Merged

✨ Coalesce #1360

merged 3 commits into from
Feb 20, 2025

Conversation

Vectorized
Copy link
Owner

Description

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.

Copy link

Gas Snapshot Comparison Report

Generated at commit : a9ca068, Compared to commit : 9298d09

Contract Name Test Name Main Gas PR Gas Diff
FixedPointMathLibTest testAbs() 701 679 -22
testAvg() 443 465 22
testAvgEdgeCase() 492 470 -22
testAvgSigned() 923 901 -22
testCbrt() 10283 10328 45
testCbrtWad() 12051 12073 22
testCbrtWadConverged() 2747 2725 -22
testCbrtWadDebug() 7361 7339 -22
testDistEdgeCases() 548 526 -22
testDivWadUp() 2942 2986 44
testDivWadUpEdgeCases() 428 451 23
testDivWadUpZeroDenominatorReverts() 3960 3982 22
testFactorial() 99213 99191 -22
testFullMulDiv() 1105 1128 23
testFullMulDivN() 896 874 -22
testFullMulDivUnchecked() 1635 1613 -22
testFullMulDivUpRevertsIfRoundedUpResultOverflowsCase2() 4534 4556 22
testInvMod() 12892 12915 23
testLambertW0WadKnownValues() 1685605 1685617 12
testLambertW0WadMonoDebug() 770645 770690 45
testLambertW0WadMonotonicallyIncreasing() 18416553 18416531 -22
testLambertW0WadMonotonicallyIncreasing2() 4035368 4035346 -22
testLambertW0WadRevertsForOutOfDomain() 57856 58122 266
testLerpUint() 6420 6398 -22
testLnWad() 2098 2054 -44
testLnWadBig() 2088 2066 -22
testLnWadOverflowReverts() 4072 4028 -44
testLnWadSmall() 2689 2667 -22
testLog10() 76212 76190 -22
testLog10Up() 4435 4413 -22
testLog2() 243209 243231 22
testLog256() 22809 22853 44
testLog256Up() 1271 1294 23
testLog2Up() 297324 297369 45
testMulDivEdgeCases() 740 741 1
testMulDivUpEdgeCases() 817 862 45
testMulDivUpZeroDenominator() 4034 4077 43
testMulDivZeroDenominatorReverts() 4033 4055 22
testMulWad() 712 690 -22
testMulWadUp() 779 757 -22
testMulWadUpEdgeCases() 776 799 23
testRPow() 3298 3276 -22
testRPowOverflowReverts() 8212 8168 -44
testSDivWad() 862 840 -22
testSMulWad() 1011 1053 42
testSaturatingAdd() 1710 1774 64
testSaturatingMul() 1815 1838 23
testSci() 1838636 1838614 -22
testSqrt() 43797 43842 45
testSqrtHashedSingle() 54632 54610 -22
testSqrtWad() 7937 7981 44
test__codesize() 49208 49829 621

@Vectorized Vectorized merged commit 6015421 into main Feb 20, 2025
13 checks passed
@Vectorized Vectorized deleted the coalesce branch February 20, 2025 05:11
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