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

Add explanation about new optimization simpleCounterForLoopUncheckedIncrement #14648

Closed
matheusaaguiar opened this issue Oct 25, 2023 · 2 comments · Fixed by ykk12345/solidity#1
Assignees
Labels
documentation 📖 selected for development It's on our short-term development
Milestone

Comments

@matheusaaguiar
Copy link
Collaborator

Page

Optimizer Section of the documentation.

https://docs.soliditylang.org/en/latest/internals/optimizer.html

Abstract

Since 0.8.22, the compiler can use unchecked arithmetic when incrementing the counter of for loops, under certain specific conditions. Those conditions need to be properly explained in the documentation.
The blog post about the 0.8.22 release and the PR that introduced the functionality have the necessary information and examples about that.

@ekpyron ekpyron added this to the 0.8.23 milestone Oct 25, 2023
@ekpyron ekpyron added must have Something we consider an essential part of Solidity 1.0. selected for development It's on our short-term development and removed must have Something we consider an essential part of Solidity 1.0. labels Oct 25, 2023
@ekpyron
Copy link
Member

ekpyron commented Oct 25, 2023

This will in fact open a third subsection there besides Opcode-Based Optimizer Module and Yul-Based Optimizer Module.
As a name for that section, I'd suggest Codegen-Based Optimizations or Optimized Code Generation or such, but open for better choices (Code Generation Optimizations Based On An Analysis Of Solidity Code is a bit bulky :-)).

@matheusaaguiar
Copy link
Collaborator Author

Closed by #14650.

@github-project-automation github-project-automation bot moved this from In Progress to Done in Solidity Focus Board Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation 📖 selected for development It's on our short-term development
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants