only capture time when not cached

time is also expensive...
This commit is contained in:
Tobias Koppers 2021-10-06 10:12:48 +02:00
parent 283359261d
commit 7311f955a8
1 changed files with 7 additions and 7 deletions

View File

@ -215,7 +215,6 @@ const visitModules = (
}
let blockModules = blockModulesMap.get(block);
if (blockModules !== undefined) return blockModules;
logger.time("visitModules: prepare");
const module = /** @type {Module} */ (block.getRootBlock());
const memCache = moduleMemCaches && moduleMemCaches.get(module);
if (memCache !== undefined) {
@ -223,22 +222,23 @@ const visitModules = (
"bundleChunkGraph.blockModules",
runtime,
() => {
logger.time("visitModules: prepare");
const map = new Map();
extractBlockModules(module, moduleGraph, runtime, map);
logger.timeAggregate("visitModules: prepare");
return map;
}
);
for (const [block, blockModules] of map)
blockModulesMap.set(block, blockModules);
return map.get(block);
} else {
logger.time("visitModules: prepare");
extractBlockModules(module, moduleGraph, runtime, blockModulesMap);
blockModules = blockModulesMap.get(block);
logger.timeAggregate("visitModules: prepare");
return blockModules;
}
logger.timeAggregate("visitModules: prepare");
blockModules = blockModulesMap.get(block);
if (blockModules === undefined) {
blockModulesMap.set(block, (blockModules = []));
}
return blockModules;
};
let statProcessedQueueItems = 0;