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

✨ invMod #1183

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

✨ invMod #1183

wants to merge 1 commit into from

Conversation

Vectorized
Copy link
Owner

Description

This one is actually very fast.

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 : 88b1049, Compared to commit : f4b41ee

Contract Name Test Name Main Gas PR Gas Diff
FixedPointMathLibTest testAvgEdgeCase() 448 492 44
testCbrtWadConverged() 2748 2726 -22
testCbrtWadDebug() 7360 7338 -22
testDist() 615 678 63
testDistEdgeCases() 527 505 -22
testDivWadUp() 2964 2942 -22
testDivWadUpEdgeCases() 450 428 -22
testFullMulDiv() 1149 1127 -22
testFullMulDivUnchecked() 1591 1635 44
testLambertW0WadMonoDebug() 770689 770667 -22
testLambertW0WadMonotonicallyIncreasing() 18416531 18416509 -22
testLambertW0WadMonotonicallyIncreasing2() 4035302 4035325 23
testLambertW0WadRevertsForOutOfDomain() 16552 16530 -22
testLerpUint() 6421 6398 -23
testLnWad() 2076 2054 -22
testLnWadNegativeReverts() 3307 3285 -22
testLnWadOverflowReverts() 3305 3283 -22
testLnWadSmall() 2688 2666 -22
testLog10() 76211 76189 -22
testLog10Up() 4413 4391 -22
testLog256() 22808 22831 23
testLog2Up() 297325 297368 43
testMulDiv() 1868 1912 44
testMulDivEdgeCases() 784 762 -22
testMulDivUpEdgeCases() 796 861 65
testMulWadEdgeCases() 731 709 -22
testMulWadUpEdgeCases() 754 799 45
testRPow() 3276 3299 23
testSMulWad() 1054 1032 -22
testSci() 1838613 1838636 23
testSqrtWad() 7959 7937 -22
test__codesize() 45535 45840 305
testInvMod() - 12892 -

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