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

LDC support in predicate feature #456

Closed
Marvin8122 opened this issue Feb 18, 2023 · 2 comments · Fixed by #612
Closed

LDC support in predicate feature #456

Marvin8122 opened this issue Feb 18, 2023 · 2 comments · Fixed by #612

Comments

@Marvin8122
Copy link

Marvin8122 commented Feb 18, 2023

LDC support

Is it actually practical or sensible to support LDC instraction? How does losing stateless verification affect us? Is it even worth having predicates anymore, if they cannot be verified statelessly?

If we decide to allow LDC, then we could likely support CCP, CROO, CSIZ as well. Maybe storage reads could be supported as well?

Originally posted by @Dentosal in #440 (comment)

@xgreenx xgreenx changed the title There has been discussion about not having a fixed gas limit, and instead having the predicate either share the gas limit with the primary transaction, or to have a separate gas limit. LDC support in predicate feature May 26, 2023
@Voxelot
Copy link
Member

Voxelot commented Jun 5, 2023

Also consider other opcodes related to contracts that might be useful, such as code size

@SilentCicero
Copy link
Member

Related: #492

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 a pull request may close this issue.

3 participants