Closed Bug 1415486 Opened 2 years ago Closed 2 years ago

Nursery allocate async functions

Categories

(Core :: JavaScript Engine, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: jandem, Assigned: jandem)

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
This seems to just work. Improves the micro-benchmark below from 418 ms to 84 ms.

function f() {
    arr = [];
    for (var i = 0; i < 1000000; i++)
        arr.push({});
    var t = new Date;
    for (let i = 0; i < 1000000; i++) {
        g = async function() { return 1; };
    }
    print(new Date - t);
}
f();
Attachment #8926317 - Flags: review?(arai.unmht)
Comment on attachment 8926317 [details] [diff] [review]
Patch

Review of attachment 8926317 [details] [diff] [review]:
-----------------------------------------------------------------

Great :D
Attachment #8926317 - Flags: review?(arai.unmht) → review+
Pushed by jandemooij@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c34360f7ff67
Nursery allocate async functions. r=arai
https://hg.mozilla.org/mozilla-central/rev/c34360f7ff67
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.