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.

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

Support nested objects for Yul tests. #8143

Open
chriseth opened this issue Jan 15, 2020 · 0 comments
Open

Support nested objects for Yul tests. #8143

chriseth opened this issue Jan 15, 2020 · 0 comments
Labels
low effort There is not much implementation work to be done. The task is very easy or tiny. low impact Changes are not very noticeable or potential benefits are limited. must have eventually Something we consider essential but not enough to prevent us from releasing Solidity 1.0 without it. testing 🔨

Comments

@chriseth
Copy link
Contributor

This needs to extend the analysis to recurse into the object in test/libyul/Common.cpp .
@chriseth chriseth mentioned this issue Jan 15, 2020
Merged
@cameel cameel added low effort There is not much implementation work to be done. The task is very easy or tiny. medium impact Default level of impact must have eventually Something we consider essential but not enough to prevent us from releasing Solidity 1.0 without it. low impact Changes are not very noticeable or potential benefits are limited. and removed medium impact Default level of impact labels Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
low effort There is not much implementation work to be done. The task is very easy or tiny. low impact Changes are not very noticeable or potential benefits are limited. must have eventually Something we consider essential but not enough to prevent us from releasing Solidity 1.0 without it. testing 🔨
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@cameel @nikola-matic @chriseth