nprasad2077/oxc
1
0 Fork 0
mirror of https://github.com/oxc-project/oxc.git synced 2026-11-04 10:29:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

perf(transformer): arrow function transform more efficient scope search (#5842)

Browse Source 
Micro-optimization. Match scopes with a single comparison, instead of 2.
This commit is contained in:
overlookmotel committed 2024-09-18 02:23:37 +00:00
1 parent 03e02a0eac
commit fd70c4b885
1 file changed
+10 -6
Download Patch File Download Diff File Expand all files Collapse all files
crates/oxc_transformer/src/es2015/arrow_functions.rs
+10 -6
View File
@@ -252,16 +252,20 @@ impl<'a> ArrowFunctions<'a> {
// <https://github.com/oxc-project/oxc/pull/5840>
let this_var = self.this_var_stack.last_mut().unwrap();
if this_var.is_none() {
let target_scope_id =
ctx.scopes().ancestors(ctx.current_scope_id()).skip(1).find(|&scope_id| {
let target_scope_id = ctx
.scopes()
.ancestors(ctx.current_scope_id())
.skip(1)
.find(|&scope_id| {
let scope_flags = ctx.scopes().get_flags(scope_id);
scope_flags.intersects(ScopeFlags::Function | ScopeFlags::Top)
&& !scope_flags.contains(ScopeFlags::Arrow)
});
// Function but not arrow function
scope_flags & (ScopeFlags::Function | ScopeFlags::Arrow) == ScopeFlags::Function
})
.unwrap_or(ctx.scopes().root_scope_id());
this_var.replace(BoundIdentifier::new_uid(
"this",
target_scope_id.unwrap(),
target_scope_id,
SymbolFlags::FunctionScopedVariable,
ctx,
));