beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 07:30:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

LibWasm: Stop using ConstrainedStream for function parsing

Browse source 
Speeds up spidermonkey.wasm instantiation by around 20ms (260ms -> 240ms)
This commit is contained in:
Diego Frias 2024-07-26 13:46:31 -07:00 committed by Andreas Kling
parent 5cde327d46
commit 2192c149e2
Notes: github-actions[bot] 2024-07-27 06:20:49 +00:00 
Author: https://github.com/dzfrias
Commit: https://github.com/LadybirdBrowser/ladybird/commit/2192c149e29
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/850
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/LibWasm/Parser/Parser.cpp
View file

@ -1257,11 +1257,9 @@ ParseResult<CodeSection::Code> CodeSection::Code::parse(Stream& stream)
return with_eof_check(stream, ParseError::InvalidSize); return with_eof_check(stream, ParseError::InvalidSize);
size_t size = size_or_error.release_value(); size_t size = size_or_error.release_value();
auto constrained_stream = ConstrainedStream { MaybeOwned<Stream>(stream), size };
// Emprically, if there are `size` bytes to be read, then there's around // Emprically, if there are `size` bytes to be read, then there's around
// `size / 2` instructions, so we pass that as our size hint. // `size / 2` instructions, so we pass that as our size hint.
auto func = TRY(Func::parse(constrained_stream, size / 2)); auto func = TRY(Func::parse(stream, size / 2));
return Code { static_cast<u32>(size), func }; return Code { static_cast<u32>(size), func };
} }