Skip to content

Conversation

Dentosal
Copy link
Member

Description

Heavily WIP. After new load/store ops for u16/u32 (from FuelLabs/fuel-specs#628) it would be neat to pack the data section by removing padding in favor of alingmnet, and performing substring search when inserting new constants.

Checklist

  • I have linked to any relevant issues.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have updated the documentation where relevant (API docs, the reference, and the Sway book).
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
  • I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
  • I have requested a review from the relevant team or maintainers.

@Dentosal Dentosal self-assigned this Apr 14, 2025
@Dentosal
Copy link
Member Author

Dentosal commented Jul 3, 2025

Closing in favor of doing this in multiple smaller PRs, first of which is #7271

@Dentosal Dentosal closed this Jul 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant