Closed Bug 504590 Opened 11 years ago Closed 11 years ago
Testcase confirmed on Windows Vista. Regression range: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=d056669074c7&tochange=57213af4a45d
This is fallout from upvar2. I think we should fix it on the 1.9.1 branch. But it doesn't block.
The closure forces the inner function to be HEAVYWEIGHT (since it's very hard for the compiler to clear the flag) and we end up falling into the non-null-closure case of JSOP_LAMBDA. That branch avoids cloning if the compiler saw the right parent (which it does in this case) but that, in effect, creates a joined function object, whose semantics we don't quite support yet (see bug 471214).
Assignee: general → mrbkap
Status: NEW → ASSIGNED
Attachment #388970 - Flags: review?(brendan)
Comment on attachment 388970 [details] [diff] [review] Proposed fix Any perf effects of note? Please use more standard (cough, we have two leading forms) FIXME style, preferably FIXME: bug NNNNNN. /be
Attachment #388970 - Flags: review?(brendan) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.