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

Use only a single stack slot for storage references. #2460

Merged
merged 1 commit into from Jul 14, 2015

Conversation

Projects
None yet
3 participants
@chriseth
Contributor

chriseth commented Jul 14, 2015

Only struct or array types can persistently reference storage. Those types always align on a full stack slot, so it is not necessary to store the intra-slot byte offset. This doubles the possible number of local storage variables.

@arkpar arkpar added looksgood and removed pleasereview labels Jul 14, 2015

gavofyork pushed a commit that referenced this pull request Jul 14, 2015

Gav Wood
Merge pull request #2460 from chriseth/sol_singleSlotForStructRef
Use only a single stack slot for storage references.

@gavofyork gavofyork merged commit 6ad7936 into ethereum:develop Jul 14, 2015

5 of 7 checks passed

Linux C++ evmjit pull requests DEV build done.
Details
OSX C++ evmjit pull requests DEV build done.
Details
Linux C++ develop check DEV build done.
Details
Linux C++ pull requests DEV build done.
Details
OSX C++ develop check DEV build done.
Details
OSX C++ pull requests DEV build done.
Details
Windows C++ pull requests DEV build done.
Details

@gavofyork gavofyork removed the in progress label Jul 14, 2015

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