Closed Bug 1604179 Opened 5 years ago Closed 5 years ago

Initialize more ImmutableFlags on LazyScripts

Categories

(Core :: JavaScript Engine, task, P2)

task

Tracking

()

RESOLVED FIXED
mozilla73
Tracking Status
firefox73 --- fixed

People

(Reporter: tcampbell, Assigned: tcampbell)

References

Details

Attachments

(5 files)

In Bug 1604064 we add a whitelist of flags that are not set for LazyScripts. This bug tries to reduce the number of exceptions so that LazyScript and JSScript are more predictable.

Attachment #9116121 - Attachment description: Bug 1604179 - Initialize IsFunction flag for LazyScripts → Bug 1604179 - Initialize IsFunction flag for LazyScripts. r?jandem
Attachment #9116122 - Attachment description: Bug 1604179 - Initialize FunHasExtensibleScope flag for LazyScripts → Bug 1604179 - Initialize FunHasExtensibleScope flag for LazyScripts. r?jandem
Attachment #9116124 - Attachment description: Bug 1604179 - Initialize HasMappedArgsObj flag for LazyScripts → Bug 1604179 - Initialize HasMappedArgsObj flag for LazyScripts. r?jandem
Attachment #9116123 - Attachment description: Bug 1604179 - Initialize HasCallSiteObj flag for LazyScripts → Bug 1604179 - Initialize HasCallSiteObj flag for LazyScripts. r?jandem
Attachment #9116125 - Attachment description: Bug 1604179 - Initialize ArgumentsHasVarsBinding flag for LazyScripts → Bug 1604179 - Initialize ArgumentsHasVarsBinding flag for LazyScripts. r?jandem
Pushed by tcampbell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8ad0fbbb4f47 Initialize IsFunction flag for LazyScripts. r=jandem https://hg.mozilla.org/integration/autoland/rev/d77ba1957fa6 Initialize FunHasExtensibleScope flag for LazyScripts. r=jandem https://hg.mozilla.org/integration/autoland/rev/f773a6310617 Initialize HasMappedArgsObj flag for LazyScripts. r=jandem https://hg.mozilla.org/integration/autoland/rev/e862f96814e9 Initialize HasCallSiteObj flag for LazyScripts. r=jandem https://hg.mozilla.org/integration/autoland/rev/69ce37506e86 Initialize ArgumentsHasVarsBinding flag for LazyScripts. r=jandem
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: