-
-
Notifications
You must be signed in to change notification settings - Fork 804
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
feat: optimize dynarray and bytearray copies #3499
feat: optimize dynarray and bytearray copies #3499
Conversation
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. @@ Coverage Diff @@
## master #3499 +/- ##
==========================================
- Coverage 89.19% 89.16% -0.04%
==========================================
Files 84 84
Lines 10948 10953 +5
Branches 2503 2503
==========================================
+ Hits 9765 9766 +1
- Misses 771 775 +4
Partials 412 412
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
include the length word store into the batch copy
batch the length copy into the copy
1232cba
to
bcb94e2
Compare
What I did
include the length word in the batch copy instead of issuing a separate store instruction.
brings CurveStableSwapMetaNG.vy down by 315 bytes (~1.5%) and VaultV3.vy by 45 bytes (0.25%) in both
--optimize codesize
and--optimize gas
modes.How I did it
How to verify it
Commit message
Description for the changelog
Cute Animal Picture