mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-04-24 13:35:12 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibJS: Elide empty variable environment for default parameters

Browse source 
No need to create a separate variable environment if all the parameters
are going into locals anyway.
This commit is contained in:
Andreas Kling 2025-03-20 17:37:48 -05:00
parent 8be966524f
commit 24c2a9de6f
1 changed files with 13 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
Libraries/LibJS/Bytecode/Generator.cpp
View file

@ -136,7 +136,19 @@ CodeGenerationErrorOr<void> Generator::emit_function_declaration_instantiation(E
}
}
} else {
emit<Op::CreateVariableEnvironment>(function.m_var_environment_bindings_count);
bool has_non_local_parameters = false;
if (scope_body) {
for (auto const& variable_to_initialize : function.m_var_names_to_initialize_binding) {
auto const& id = variable_to_initialize.identifier;
if (!id.is_local()) {
has_non_local_parameters = true;
break;
}
}
}
if (has_non_local_parameters)
emit<Op::CreateVariableEnvironment>(function.m_var_environment_bindings_count);
if (scope_body) {
for (auto const& variable_to_initialize : function.m_var_names_to_initialize_binding) {