Bug 445262 (upvar1)

display closure optimizations for SpiderMonkey, part I

RESOLVED FIXED in mozilla1.9.1a2

Status

()

Core
JavaScript Engine
P1
normal
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: brendan, Assigned: brendan)

Tracking

Trunk
mozilla1.9.1a2
Points:
---
Dependency tree / graph
Bug Flags:
wanted1.9.1 +
in-testsuite -
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

10 years ago
Patch forthcoming, inspired by Chez Scheme. Need to get it working and measure a bit first.

/be
The alpha freezes today: is that still a reasonable expectation for this patch arriving?  We're dying for it over in tracemonkey.
(Assignee)

Comment 2

10 years ago
It's not coming today, I'm on a plane to Oslo via London. Don't cut anything off at the knees, though.

/be
No knee-cutting, just eagerness. :)
(Assignee)

Updated

10 years ago
Status: NEW → ASSIGNED
Priority: -- → P1
(Assignee)

Updated

10 years ago
Blocks: 446379

Updated

10 years ago
Flags: wanted1.9.1?

Updated

10 years ago
Flags: wanted1.9.1? → wanted1.9.1+

Updated

10 years ago
Target Milestone: mozilla1.9.1a1 → mozilla1.9.1a2
(Assignee)

Comment 4

10 years ago
Some of this landed, just what I called "v0.1": not display closures but displays:

http://hg.mozilla.org/tracemonkey/index.cgi/rev/5bc82976d48b
http://hg.mozilla.org/tracemonkey/index.cgi/rev/63ba972636ad
http://hg.mozilla.org/tracemonkey/index.cgi/rev/b908f7a072d4

I should maybe clone this bug and morph it to cover the above, and make the clone be about display closures for real. Need sleep first.

/be
(Assignee)

Comment 5

10 years ago
Part II is bug 452498. Closing this since it has a landed patch, one patch per bug and all that.

/be
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Summary: display closure optimizations for SpiderMonkey → display closure optimizations for SpiderMonkey, part I

Updated

10 years ago
Depends on: 451884

Updated

10 years ago
Blocks: 452498

Updated

10 years ago
Depends on: 451721

Updated

10 years ago
No longer depends on: 451721

Updated

10 years ago
Flags: in-testsuite-
Flags: in-litmus-
(Assignee)

Updated

9 years ago
No longer depends on: 487967
(Assignee)

Updated

9 years ago
Alias: upvar1
(Assignee)

Updated

9 years ago
Depends on: 487967
You need to log in before you can comment on or make changes to this bug.