dan/oxc
1
0
Fork 0
mirror of https://github.com/danbulant/oxc synced 2026-05-19 04:08:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

perf(semantic): do not record ast nodes for cfg if cfg disabled (#4263)

Browse source 
Control flow graph builder records AST node IDs in a temp structure `ast_node_records`. Disable this if CFG is disabled.
This commit is contained in:
overlookmotel 2024-07-15 04:31:47 +00:00
parent da69076c98
commit 23743dbd59
1 changed files with 13 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
crates/oxc_semantic/src/builder.rs
View file

@ -250,18 +250,26 @@ impl<'a> SemanticBuilder<'a> {
}
fn record_ast_nodes(&mut self) {
self.ast_node_records.push(AstNodeId::dummy());
if self.cfg.is_some() {
self.ast_node_records.push(AstNodeId::dummy());
}
}
#[allow(clippy::unnecessary_wraps)]
fn retrieve_recorded_ast_node(&mut self) -> Option<AstNodeId> {
Some(self.ast_node_records.pop().expect("there is no ast node record to stop."))
if self.cfg.is_some() {
Some(self.ast_node_records.pop().expect("there is no ast node record to stop."))
} else {
None
}
}
fn record_ast_node(&mut self) {
if let Some(record) = self.ast_node_records.last_mut() {
if *record == AstNodeId::dummy() {
*record = self.current_node_id;
if self.cfg.is_some() {
if let Some(record) = self.ast_node_records.last_mut() {
if *record == AstNodeId::dummy() {
*record = self.current_node_id;
}
}
}
}