microsoft/TypeScript
1
0
Fork 0
mirror of https://github.com/microsoft/TypeScript.git synced 2026-04-17 12:32:42 -05:00
Code Issues Packages Projects Releases 212 Wiki Activity

Store node children WeakMaps in vars and bind its methods as locals.

Browse Source
This commit is contained in:
Daniel Rosenwasser
2024-06-06 06:15:54 +00:00
parent c325b103d0
commit 9957999fb5
1 changed files with 10 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/compiler/factory/nodeChildren.ts
View File

@@ -4,18 +4,25 @@ import {
Node,
} from "../_namespaces/ts.js";
const nodeChildren = new WeakMap<Node, readonly Node[] | undefined>();
// Why var? It avoids TDZ checks in the runtime which can be costly.
// See: https://github.com/microsoft/TypeScript/issues/52924
/* eslint-disable no-var */
var nodeChildren = new WeakMap<Node, readonly Node[] | undefined>();
var nodeChildrenGet = nodeChildren.get.bind(nodeChildren);
var nodeChildrenSet = nodeChildren.set.bind(nodeChildren);
/* eslint-enable no-var */
/** @internal */
export function getNodeChildren(node: Node): readonly Node[] | undefined {
if (!isNodeKind(node.kind)) return emptyArray;
return nodeChildren.get(node);
return nodeChildrenGet(node);
}
/** @internal */
export function setNodeChildren(node: Node, children: readonly Node[]): readonly Node[] {
nodeChildren.set(node, children);
nodeChildrenSet(node, children);
return children;
}