system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-10-07 00:19:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

LibJS: Remove expensive dead code in get_source_range

Browse source 
This was forgotten when cleaning up debug prints but has a huge
performance impact (around 1s / 15% on test-js).
This commit is contained in:
Simon Wanner 2023-11-04 16:39:10 +01:00 committed by Andreas Kling
parent 7a83870905
commit 969d9e1fd3
1 changed files with 1 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
Userland/Libraries/LibJS/Runtime/VM.cpp
View file

@ -1174,10 +1174,8 @@ static Optional<UnrealizedSourceRange> get_source_range(ExecutionContext const*
// JIT frame
for (auto address : native_stack) {
auto range = native_executable->get_source_range(*context->executable, address);
if (range.has_value()) {
auto realized = range->realize();
if (range.has_value())
return range;
}
}
return {};