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

feat: allow bytes substitution for uint64 template vars #306

Merged

joe-p merged 2 commits into main from feat/bytes_int_var

Aug 9, 2024

Contributor

joe-p commented Aug 9, 2024 •

edited

Loading

Proposed Changes

The AVM compresses integers meaning uint64 template variables might have an unknown width when used with pushint. This PR allows the given TEAL to use pushbytes or bytes for uint64 template variables which will preserve the integer width. This will allow PC mapping to stay consistent between initial compile time and deployment.

For an example of how a compiler might output TEAL, see https://github.com/algorandfoundation/TEALScript/pull/522/files

Assuming no objections, a similar PR will need to be made for py as well

joe-p added 2 commits

August 9, 2024 07:07


          feat: allow bytes substitution for uint64 template vars

915cb1e


          docs

d1576e8

robdmoore approved these changes

View reviewed changes

joe-p merged commit 138ae68 into main

2 checks passed

joe-p deleted the feat/bytes_int_var branch

August 9, 2024 11:28

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

Labels

None yet