PR #86110 [turbopack] Remove `Clone` from `Effect`

[turbopack] Remove `Clone` from `Effect` #86110

lukesandberg merged 8 commits into canary from closures

ijjk added created-by: Turbopack team

ijjk added Turbopack

lukesandberg changed the title ~~Add a debug assertion that we do not produce duplicate code gens.~~ [turbopack] Add a debug assertion that we do not produce duplicate code gens. 181 days ago

lukesandberg changed the title ~~[turbopack] Add a debug assertion that we do not produce duplicate code gens.~~ [turbopack] Remove `Clone` from `Effect` 181 days ago

Add a debug assertion that we do not produce duplicate code gens.

f3aa1aa0

Remove `clone` from `Effect`

8970d451

avoid linking args multiple times in the case of alternates

7083348e

make the debug assertions conditional

da96e23e

lukesandberg force pushed to da96e23e 180 days ago

lukesandberg marked this pull request as ready for review 180 days ago

lukesandberg requested a review 180 days ago

graphite-app commented on 2025-11-14

Fix a bug in how unreachable effects were created for finally blocks

557c3f7d

mischnic approved these changes on 2025-11-14

clippy!

454a043d

fix release build

df923421

try moving handle_well_known_function_call to improve diff alignment

89d47600

mischnic approved these changes on 2025-11-14

lukesandberg merged a3aadbd3 into canary 178 days ago

lukesandberg deleted the closures branch 178 days ago

github-actions added locked

github-actions locked as resolved and limited conversation to collaborators 163 days ago

Reviewers

mischnic

graphite-app

Assignees

No one assigned

Labels

Turbopack created-by: Turbopack team locked

Milestone

No milestone

next.js [turbopack] Remove `Clone` from `Effect` #86110 Merged

[turbopack] Remove `Clone` from `Effect` #86110

next.js
[turbopack] Remove `Clone` from `Effect`
#86110

Merged