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

Summary for uniswapV2LibraryGetAmountOut and uniswapV2LibraryGetAmountIn #35

Merged
merged 4 commits into from
Sep 23, 2024

Conversation

Robertorosmaninho
Copy link
Contributor

@Robertorosmaninho Robertorosmaninho commented Sep 19, 2024

This PR adds two new summarized rules regarding program execution. The rule presented in this PR summarizes the uniswapV2LibraryGetAmountOut and uniswapV2LibraryGetAmountIn functions.

By summarizing the uniswapV2LibraryGetAmountOut function, we save 380 steps.
(Without it, the program took 26748, and after implementing it, the program took 26368)

By summarizing the uniswapV2LibraryGetAmountIn function, we save 408 steps.
(Without it, the program took 26368, and after implementing it, the program took 25960)

@Robertorosmaninho Robertorosmaninho changed the title Summary for uniswapV2LibraryGetAmountOut Summary for uniswapV2LibraryGetAmountOut and uniswapV2LibraryGetAmountIn Sep 19, 2024
Base automatically changed from sorttokens-summary to main September 19, 2024 18:47
@Robertorosmaninho Robertorosmaninho marked this pull request as ready for review September 19, 2024 19:00
@Robertorosmaninho Robertorosmaninho merged commit b0fe354 into main Sep 23, 2024
1 check passed
@Robertorosmaninho Robertorosmaninho deleted the getamountout-summary branch September 23, 2024 15:33
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.

3 participants