eliott/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird synced 2026-05-05 06:32:30 +02:00
Code Issues Packages Projects Releases Wiki Activity

LibJS: Remove FLATTEN from bytecode interpreter

Browse Source 
This doesn't appear to improve performance on my machine anymore.
It also very modestly reduces interpreter size.
This commit is contained in:
Andreas Kling
2025-12-04 11:25:57 +01:00
committed by Andreas Kling
parent fca29400af
commit d2f9f91e45
Notes: github-actions[bot] 2025-12-10 03:47:22 +00:00 
Author: https://github.com/awesomekling
Commit: https://github.com/LadybirdBrowser/ladybird/commit/d2f9f91e45d
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/7080
1 changed files with 1 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
Libraries/LibJS/Bytecode/Interpreter.cpp
View File

@@ -251,14 +251,7 @@ NEVER_INLINE Interpreter::HandleExceptionResponse Interpreter::handle_exception(
VERIFY_NOT_REACHED();
}
// FIXME: GCC takes a *long* time to compile with flattening, and it will time out our CI. :|
#if defined(AK_COMPILER_CLANG)
# define FLATTEN_ON_CLANG FLATTEN
#else
# define FLATTEN_ON_CLANG
#endif
FLATTEN_ON_CLANG void Interpreter::run_bytecode(size_t entry_point)
void Interpreter::run_bytecode(size_t entry_point)
{
if (vm().did_reach_stack_space_limit()) [[unlikely]] {
reg(Register::exception()) = vm().throw_completion<InternalError>(ErrorType::CallStackSizeExceeded).value();