Open Bug 1657940 Opened 5 years ago Updated 10 months ago

Consider lowering StringToAtomCache::MinStringLength

Categories

(Core :: JavaScript Engine, task, P3)

task

Tracking

()

People

(Reporter: jandem, Unassigned)

References

(Blocks 2 open bugs)

Details

Bug 1657559 is adding the cache for strings with length >= 30, because shorter strings aren't as expensive to hash. However for strings with length 29, the cache would still be a win so we should consider lowering it. Maybe to 20-25 or so.

This would also take some pressure off string deduplication happening during nursery GC because we will be able to use the StringToAtomCache for more strings.

You need to log in before you can comment on or make changes to this bug.