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

@inline decorator #4521

Open
cyberthirst opened this issue Mar 13, 2025 · 1 comment
Open

@inline decorator #4521

cyberthirst opened this issue Mar 13, 2025 · 1 comment
Labels
release - tentative items still being considered for release inclusion
Milestone
v0.4.2

Comments

@cyberthirst
Copy link
Collaborator

cyberthirst commented Mar 13, 2025 •
edited
Loading

Simple Summary

Add a new @inline decorator to manually control which functions get inlined.

Motivation

Allow users more granular control over gas/codesize tradeoff.

Specification

@inline decorator marks a function to be inlined. It's a noop for the legacy pipeline and only actuates with Venom.

Considerations

Also add @noinline decorator to control the tradeoff in the opposite direction.
@cyberthirst cyberthirst added the needs triage needs triage label Mar 13, 2025
@cyberthirst cyberthirst added this to the v0.4.2 milestone Mar 13, 2025
@cyberthirst
Copy link
Collaborator Author

draft implementation: #4481

@charles-cooper charles-cooper added release - tentative items still being considered for release inclusion and removed needs triage needs triage labels Mar 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
release - tentative items still being considered for release inclusion
Projects
None yet
Milestone
v0.4.2
Development

No branches or pull requests
2 participants
@charles-cooper @cyberthirst