Closed Bug 1881995 Opened 7 months ago Closed 5 months ago

Replace strings during atomization with a new string type which points to the corresponding atom

Categories

(Core :: JavaScript Engine, task, P2)

task

Tracking

()

RESOLVED FIXED
127 Branch
Tracking Status
firefox127 --- fixed

People

(Reporter: alexical, Assigned: alexical)

References

(Depends on 1 open bug, Blocks 3 open bugs, Regressed 1 open bug)

Details

(Whiteboard: [sp3])

Attachments

(1 file)

Chromium does something similar. Basically when we atomize a string, in many cases we can replace that string in-line with a new string type which holds a pointer to the corresponding atom. Then, that atom can be cheaply looked up either in C++ or JIT code.

Assignee: nobody → dothayer
Status: NEW → ASSIGNED
Whiteboard: [sp3]
Priority: -- → P2
Blocks: 1889509
Blocks: 1889510
Depends on: 1890394
Pushed by dothayer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ae6126131b6d
Implement ForwardedAtoms and create them during atomization r=iain,sfink

Please land this after the next merge, especially now with the long weekend coming up :)

Pushed by dothayer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fa120d93961e
Implement ForwardedAtoms and create them during atomization r=iain,sfink
Flags: needinfo?(dothayer)
Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 127 Branch
Regressions: 1895678
Blocks: 1900142
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: