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.

Sign up for GitHub

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

Jump to bottom

opti: use simple div by 32 instead of divmod when using ceil32 #613

Open

obatirou opened this issue Jan 30, 2025 · 0 comments

Milestone

Contributor

obatirou commented Jan 30, 2025 •

edited by Eikix

Loading

Following #612
The codebase should use simple div by 32 instead of divmod when using ceil3

How:

use native cairo division when we are sure the number being divided is a multiple
look for all calls to divmod and unsigned_div_rem, if the number being divided is a multiple of the divider, then replace by native cairo operation and document it

obatirou added this to Keth

github-project-automation bot moved this to Backlog in Keth

obatirou mentioned this issue

feat: identity precompile #612

Merged

Eikix moved this from Backlog to Todo in Keth

Eikix added this to the Optimization milestone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment