Skip to content

Commit ddc7fa2

Browse files
committed
remove the turbo task from apply_reexport_tree_shaking
This got exactly one cache hit (which makes me think a module has a duplicate export...) and that makes sense. mostly this should be a quick exit and when it does call `follow_reexports` we can rely on the cache there.
1 parent 62bd65b commit ddc7fa2

File tree

1 file changed

+2
-1
lines changed
  • turbopack/crates/turbopack/src

1 file changed

+2
-1
lines changed

turbopack/crates/turbopack/src/lib.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -205,12 +205,14 @@ async fn apply_module_type(
205205
part,
206206
side_effect_free_packages,
207207
)
208+
.await?
208209
} else {
209210
apply_reexport_tree_shaking(
210211
Vc::upcast(*module),
211212
part,
212213
side_effect_free_packages,
213214
)
215+
.await?
214216
}
215217
}
216218
_ => bail!(
@@ -280,7 +282,6 @@ async fn apply_module_type(
280282
.cell())
281283
}
282284

283-
#[turbo_tasks::function]
284285
async fn apply_reexport_tree_shaking(
285286
module: Vc<Box<dyn EcmascriptChunkPlaceable>>,
286287
part: ModulePart,

0 commit comments

Comments
 (0)