Closed
Bug 1217099
Opened 9 years ago
Closed 9 years ago
Stop emitting pointless JSOP_GETLOCAL; JSOP_POP bytecode sequence for `var x;`
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla44
Tracking | Status | |
---|---|---|
firefox44 | --- | fixed |
People
(Reporter: jorendorff, Assigned: jorendorff)
References
Details
Attachments
(1 file, 1 obsolete file)
We've been doing this forever. With the emitVariables() and emitNormalFor() refactoring in bug 1217001, we can stop.
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8677000 -
Flags: review?(shu)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → jorendorff
Status: NEW → ASSIGNED
Comment 2•9 years ago
|
||
This should fix bug 320887, right?
Assignee | ||
Comment 3•9 years ago
|
||
Yes. Good grief.
Assignee | ||
Comment 4•9 years ago
|
||
Attachment #8677562 -
Flags: review?(shu)
Assignee | ||
Updated•9 years ago
|
Attachment #8677000 -
Attachment is obsolete: true
Attachment #8677000 -
Flags: review?(shu)
Assignee | ||
Comment 5•9 years ago
|
||
I added a test for bug 320887 to the patch.
Comment 7•9 years ago
|
||
Comment on attachment 8677562 [details] [diff] [review] Stop emitting pointless JSOP_GETLOCAL; JSOP_POP bytecode sequence for `var x;` Review of attachment 8677562 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/tests/ecma_5/Global/bug-320887.js @@ +1,1 @@ > +// `var x` should not call the getter of an existing global property. lol! :(
Attachment #8677562 -
Flags: review?(shu) → review+
Assignee | ||
Comment 8•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/bc2fa0d600192e096eb1c289b23f75410c45b744 Bug 1217099 - Stop emitting pointless JSOP_GETLOCAL; JSOP_POP bytecode sequence for `var x;`. r=shu.
Comment 9•9 years ago
|
||
This landed yesterday - https://hg.mozilla.org/mozilla-central/rev/bc2fa0d60019
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in
before you can comment on or make changes to this bug.
Description
•