Rename FunctionIndex to ScriptIndex, and use ScriptIndex instead of ScriptStencil
Categories
(Core :: JavaScript Engine, task, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox86 | --- | fixed |
People
(Reporter: arai, Assigned: arai)
References
Details
Attachments
(5 files)
FunctionIndex
has been an index only for function, but now initial top-level script is also indexed by FunctionIndex
, and the name doesn't reflect the usage.
it should be renamed to ScriptIndex
for clarity.
also, as a preparation for moving SourceExtent
and gcThings
out of ScriptStencil
into a new vector or hashmap with same index, code that passes ScriptStencil*
should be rewritten to pass ScriptIndex
+ CompilationStencil
.
Assignee | ||
Comment 1•4 years ago
|
||
To reflect the top-level initial script usage, it should be called ScriptIndex.
Also, to use ScriptIndex in JSScript methods, it needs to be moved out of
Stencil.h.
Assignee | ||
Comment 2•4 years ago
|
||
Depends on D100715
Assignee | ||
Comment 3•4 years ago
|
||
Depends on D100716
Assignee | ||
Comment 4•4 years ago
|
||
Depends on D100717
Assignee | ||
Comment 5•4 years ago
|
||
Depends on D100718
Comment 7•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7278737e2250
https://hg.mozilla.org/mozilla-central/rev/b92ec7237f5e
https://hg.mozilla.org/mozilla-central/rev/2259750c4d4c
https://hg.mozilla.org/mozilla-central/rev/fa57c761af9b
https://hg.mozilla.org/mozilla-central/rev/6e0e587e3c42
Description
•