Skip to content

Conversation

Dentosal
Copy link
Member

@Dentosal Dentosal commented Jul 3, 2025

Description

Requires #7085

Adds initial support for u16 and u32 items in the data section, taking the first step in using the new LQW/LHW/SQW/SHW instructions to acheve smaller output binaries. Sadly this is unable to compress constants in the code, as they require addressing changes (probbly related to #7281).

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 Jul 3, 2025
@Dentosal Dentosal added the compiler: codegen Everything to do with IR->ASM, register allocation, etc. label Jul 3, 2025
@Dentosal Dentosal temporarily deployed to fuel-sway-bot July 3, 2025 09:08 — with GitHub Actions Inactive
@Dentosal Dentosal mentioned this pull request Jul 3, 2025
8 tasks
@Dentosal Dentosal temporarily deployed to fuel-sway-bot July 3, 2025 09:18 — with GitHub Actions Inactive
Copy link

codspeed-hq bot commented Jul 3, 2025

CodSpeed Performance Report

Merging #7271 will not alter performance

Comparing dento/u16-u32-data (efd91b5) with master (74092cb)

Summary

✅ 25 untouched benchmarks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler: codegen Everything to do with IR->ASM, register allocation, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants