Skip to content

feat(chunk): merge side effect free common chunk#9167

Draft
AliceLanniste wants to merge 2 commits into
rolldown:mainfrom
AliceLanniste:merge_side_effect_free_modules
Draft

feat(chunk): merge side effect free common chunk#9167
AliceLanniste wants to merge 2 commits into
rolldown:mainfrom
AliceLanniste:merge_side_effect_free_modules

Conversation

@AliceLanniste

@AliceLanniste AliceLanniste commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

close #4788

@AliceLanniste AliceLanniste marked this pull request as draft April 20, 2026 09:34
@AliceLanniste AliceLanniste force-pushed the merge_side_effect_free_modules branch from 245a910 to bfa5cc1 Compare April 22, 2026 13:02
@AliceLanniste AliceLanniste marked this pull request as ready for review April 22, 2026 13:05
@codspeed-hq

codspeed-hq Bot commented Apr 22, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 4 untouched benchmarks
⏩ 10 skipped benchmarks1


Comparing AliceLanniste:merge_side_effect_free_modules (2cab559) with main (cc38b86)2

Open in CodSpeed

Footnotes

  1. 10 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

  2. No successful run was found on main (d38b061) during the generation of this report, so cc38b86 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@IWANABETHATGUY IWANABETHATGUY marked this pull request as draft April 22, 2026 13:32
@IWANABETHATGUY

Copy link
Copy Markdown
Member

Please at least make sure all CI passed.

@AliceLanniste AliceLanniste force-pushed the merge_side_effect_free_modules branch from 2cab559 to cbefdab Compare April 30, 2026 03:14
@AliceLanniste AliceLanniste force-pushed the merge_side_effect_free_modules branch from cbefdab to a938dab Compare May 6, 2026 08:50
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.

[Feature Request]: reduce the number of chunks by merging sideeffect free chunks

2 participants