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

Empty asm functions should be possible #1805

Open
Shvandre opened this issue Feb 13, 2025 · 3 comments
Open

Empty asm functions should be possible #1805

Shvandre opened this issue Feb 13, 2025 · 3 comments
Labels
feature: asm Tact-embedded assembly functions kind: gas! Gas consumption and fee-related things kind: language feature Intent to add a language feature

Comments

@Shvandre
Copy link
Contributor

Now it leads to compilation error on funC stage.
Now we are using NOPs, but it consumes 18 gas and does nothing :(
@novusnota novusnota added kind: gas! Gas consumption and fee-related things scope: parser Implementation of parser (src/grammar) misc: func Any issues related to FunC and removed scope: parser Implementation of parser (src/grammar) labels Feb 13, 2025
@novusnota novusnota mentioned this issue Feb 13, 2025
Open
@Gusarich
Copy link
Member

This is FunC limitation and we won't be able to properly allow empty asm functions until we go away from func

@anton-trunov
Copy link
Member 

fun foo() asm "//"

@Gusarich
Copy link
Member 

fun foo() asm "//"

yeah cool! interestingly, I never saw anybody doing this before

@anton-trunov anton-trunov added kind: language feature Intent to add a language feature feature: asm Tact-embedded assembly functions and removed misc: func Any issues related to FunC labels Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature: asm Tact-embedded assembly functions kind: gas! Gas consumption and fee-related things kind: language feature Intent to add a language feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@anton-trunov @Gusarich @novusnota @Shvandre