-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Code/contractdata data location #13723
Comments
Will |
Yes, we will transition to one keyword for all of it, the question is rather what it will be. I was rather happy with My plan was to open the discussion about the keyword to use for this now and try to settle this asap. |
The main advantage of just sticking with The main question for that is whether it's only me to whom stuff like this looks very off:
|
Some random ideas for the keyword that would avoid tying it to a specific section in the bytecode:
|
Hm... not sure any of them really convince me right away :-). It may actually be worth a thought what happened, if we instead actually just weakened the restrictions on what's That'd be straight the opposite direction of @nventuro asking for using more keywords instead, though :-). Would reusing constant really be confusing here though? |
|
Not sure I like the idea of relaxing |
Just want to note here that allowing dynamically sized immutable data into the runtime bytecode will be challenging to deal with from a formal analysis perspective, and that my life (as an author of a symbolic execution engine) would be made easier if this would wait until after EOF (where the proper separation of code and data would make this much easier to model). |
@d-xo The problem is that currently nobody knows if and when EOF will actually happen :-). |
|
What
Introduce
code
orcontractdata
as a new data location for data stored in the deployed bytecode as an extension of the current immutable mechanism.Why
Lifting the restrictions of the current immutable mechanism and transfer the merits to dynamic types.
How
Notes
Resources
The text was updated successfully, but these errors were encountered: