forked from mirrors/gecko-dev
Bug 1898461 - Tweak our pretenuring parameters a little r=jandem
This reduces the attention threshold to 200 and increases the limit on the number of allocation sites per minor GC to 600. This shows some large high confidence improvements on jetstream2 splay subtests and some low confidence speedometer3 subtest improvements on linux. Differential Revision: https://phabricator.services.mozilla.com/D211339
This commit is contained in:
parent
5f288a0bfc
commit
4ca30b8884
1 changed files with 2 additions and 2 deletions
|
|
@ -26,13 +26,13 @@ using namespace js::gc;
|
||||||
// site. This must be large enough to ensure we have enough information to infer
|
// site. This must be large enough to ensure we have enough information to infer
|
||||||
// the lifetime and also large enough to avoid pretenuring low volume allocation
|
// the lifetime and also large enough to avoid pretenuring low volume allocation
|
||||||
// sites.
|
// sites.
|
||||||
static constexpr size_t NormalSiteAttentionThreshold = 500;
|
static constexpr size_t NormalSiteAttentionThreshold = 200;
|
||||||
static constexpr size_t UnknownSiteAttentionThreshold = 30000;
|
static constexpr size_t UnknownSiteAttentionThreshold = 30000;
|
||||||
|
|
||||||
// The maximum number of alloc sites to create between each minor
|
// The maximum number of alloc sites to create between each minor
|
||||||
// collection. Stop tracking allocation after this limit is reached. This
|
// collection. Stop tracking allocation after this limit is reached. This
|
||||||
// prevents unbounded time traversing the list during minor GC.
|
// prevents unbounded time traversing the list during minor GC.
|
||||||
static constexpr size_t MaxAllocSitesPerMinorGC = 500;
|
static constexpr size_t MaxAllocSitesPerMinorGC = 600;
|
||||||
|
|
||||||
// The maximum number of times to invalidate JIT code for a site. After this we
|
// The maximum number of times to invalidate JIT code for a site. After this we
|
||||||
// leave the site's state as Unknown and don't pretenure allocations.
|
// leave the site's state as Unknown and don't pretenure allocations.
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue