Closed Bug 1526324 Opened 5 years ago Closed 5 years ago

Stop serializing JSScript::needsArgsObj in XDR

Categories

(Core :: JavaScript Engine, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: tcampbell, Assigned: tcampbell)

References

Details

Attachments

(1 file)

After the split of script flags into mutable and immutable, it becomes clear that needsArgsObj flag is the only mutable flag that XDRScript preserves. To simplify code and remove special cases we should remove this as no other performance hinting is preserved in XDR.

DRScript should only be preserving ImmutableFlags for consistency.
While preserving the flag might have helped in some cases, it was a
special case of preserving performance hints and should be removed for
simplicity.

Pushed by tcampbell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d8e74c762135
Stop transcoding NeedsArgsObj flag in XDRScript r=jandem
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
See Also: → 1528964
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: