next.js
f7b559a6 - Remove `is_export_missing` as a turbo task

Commit
145 days ago
Remove `is_export_missing` as a turbo task In a build of vercel-site we see 70991 executions and 163683 cache hits which implies each execution gets 2.3 cache hits. By removing turbo tasks we instead have this logic execute in the parent task `EsmAssetReference::resolve_reference` Based on examining a trace of vercel-site i see that this task takes around 0.2-0.5us of cpu time, but between 4-6us of 'extent'. The `overhead.rs` benchmark tells me that the cost of a cache hit is around 220-500ns and the cost of a miss is 4.5-6us. So, in terms of latency, we can compute expected costs * as a turbotask: * lower bound: 4.5+4 + 2.3*.22 = 9.006us * upper bound: 6+6 + 2.3*.5 = 13.5us * as an `async fn` * lower bound: 3.3*4 = 13.2us * upper bound: 3.3*6 = 19.8us
Author
Committer
Parents
Loading