Bug 1745687 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
There are some redundant operations done in batch atomization during stencil instantiation
  * redundant static string lookup in `AtomizeChars` (bug 1745664)
  * `CanStoreCharsAsLatin1` check done in `NewStringCopyN`
  * redundant static string lookup `TryEmptyOrStaticString` in `NewStringCopyNDontDeflat`
  * repeated `AutoAllocInAtomsZone` in `AllocateNewAtom` that can be done in `InstantiateMarkedAtoms`
  * `Zone::isAtomsZone` branch in `JSLinearString::new_`
Revision 1 by
on 
There are some redundant operations done in batch atomization during stencil instantiation
  * redundant static string lookup in `AtomizeChars` (bug 1745664)
  * `CanStoreCharsAsLatin1` check done in `NewStringCopyN`
  * redundant static string lookup `TryEmptyOrStaticString` in `NewStringCopyNDontDeflat`
  * repeated `AutoAllocInAtomsZone` in `AllocateNewAtom` that can be done in `InstantiateMarkedAtoms`
  * `Zone::isAtomsZone` branch in `JSLinearString::new_`
  * `JSString::validateLength` operation done in `AtomizeAndCopyCharsFromLookup`

Back to Bug 1745687 Comment 0