Closed Bug 1705837 Opened 3 years ago Closed 3 years ago

Remove MakeConstructible self-hosting intrinsic

Categories

(Core :: JavaScript Engine, task, P1)

task

Tracking

()

RESOLVED FIXED
90 Branch
Tracking Status
firefox90 --- fixed

People

(Reporter: tcampbell, Assigned: tcampbell)

References

Details

Attachments

(2 files)

The MakeConstructible intrinsic is only used to make internal "Record" and "List" types work with new. This complexity can be avoided by just using something like std_Object_create directly.

Removing this helps us to avoid mutating the self-hosted templates after parsing which prevents converting the self-hosting global into a stencil.

Attachment #9216523 - Attachment description: WIP: Bug 1705837 - Add new_Record / new_List utility functions to JS self-hosting → Bug 1705837 - Add new_Record / new_List utility functions to JS self-hosting. r?jandem!
Attachment #9216524 - Attachment description: WIP: Bug 1705837 - Remove MakeConstructible self-hosting intrinsic → Bug 1705837 - Remove MakeConstructible self-hosting intrinsic. r?jandem!
Pushed by tcampbell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/075979b1470e
Add new_Record / new_List utility functions to JS self-hosting. r=jandem
https://hg.mozilla.org/integration/autoland/rev/d75eb2cd146c
Remove MakeConstructible self-hosting intrinsic. r=jandem
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: