Summary: | Make the JIT pool smaller on AS | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Filip Pizlo <fpizlo> | ||||
Component: | JavaScriptCore | Assignee: | Filip Pizlo <fpizlo> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ews-watchlist, keith_miller, mark.lam, msaboff, saam, tzagallo, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | Other | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Filip Pizlo
2021-04-30 14:07:10 PDT
Created attachment 427445 [details]
the patch
Comment on attachment 427445 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=427445&action=review > Source/JavaScriptCore/jit/ExecutableAllocator.cpp:336 > + if (Options::executablePoolSizeOverride()) > + reservation.size = Options::executablePoolSizeOverride(); this already exists right below. Just remove the ENABLE Comment on attachment 427445 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=427445&action=review > Source/JavaScriptCore/jit/ExecutableAllocator.cpp:409 > + , m_numAllocators(numberOfRegions) maybe we should rename numberOfRegions to maxNumberOfRegions *** Bug 209037 has been marked as a duplicate of this bug. *** |