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

eof: Enable eofcreate inline assembly #15981

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

rodiazet
Copy link
Contributor

@rodiazet rodiazet commented Apr 4, 2025

This PR contains:

  • Add objectName suffix of a contract name in inline assembly which maps to object name in yul.
  • Enable eofcreate in inline assembly to make eof style of contract creation possible.
  • Add and update semantic and syntaxt tests.

TODOs:

  1. How to make objectName unavailable for the contract name inside which we are using this suffix.
  2. Should it work for libraries? Edit: It looks that it can. Test added.

@rodiazet rodiazet changed the title Enable eofcreate inline assembly eof: Enable eofcreate inline assembly Apr 4, 2025
Copy link

github-actions bot commented Apr 4, 2025

Thank you for your contribution to the Solidity compiler! A team member will follow up shortly.

If you haven't read our contributing guidelines and our review checklist before, please do it now, this makes the reviewing process and accepting your contribution smoother.

If you have any questions or need our help, feel free to post them in the PR or talk to us directly on the #solidity-dev channel on Matrix.

@rodiazet rodiazet force-pushed the eofcreate-inline-assembly branch 2 times, most recently from c236ee9 to 8d2ab18 Compare April 4, 2025 12:32
…e` suffix for contract name in inline assembly for EOF
@rodiazet rodiazet force-pushed the eofcreate-inline-assembly branch from 8d2ab18 to d6e164f Compare April 4, 2025 13:13
@rodiazet rodiazet marked this pull request as draft April 4, 2025 13:31
@rodiazet rodiazet force-pushed the eofcreate-inline-assembly branch from d6e164f to a613c67 Compare April 4, 2025 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant