-
Notifications
You must be signed in to change notification settings - Fork 6
ARM Router #142
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
base: main
Are you sure you want to change the base?
ARM Router #142
Conversation
- Introduced SwapType enum for distinguishing between ARM and Wrapper swaps. - Implemented swapTokensForExactTokens function in ARMRouter. - Updated internal swap logic to handle exact token swaps. - Created unit tests for swapTokensForExactTokens functionality. - Enhanced MockWrapper with functions for price queries. - Updated shared test setup to register new configurations.
… enable gas snapshot check in foundry.toml
… next token index
…gths and next token indices
…nput checks and deposits
…ns and adjusting input checks
Comment on Gas-GolfingHere is the gas-reduction report from gas-golfing session: ExactTokensForTokens
TokensForExactTokens
ConclusionI average this is a -0.3% of gas consumption. This is good, but this comes at the expense of code smoothness. Does it makes sense to keep these changes @naddison36? |
…to external function
Gas Optimization Reduction SummaryOverviewThis report compares Initial vs Final gas costs across various swap operations. All values represent reductions (negative numbers indicate cost decreases). Detailed Breakdown
Summary Statistics
Note:
|


Description
Create a Router for ARM.