Closed
Bug 775734
Opened 12 years ago
Closed 12 years ago
remove dead hoisting code in BindVarOrConst
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla17
People
(Reporter: luke, Assigned: luke)
References
Details
(Whiteboard: [js:t])
Attachments
(2 files)
6.55 KB,
patch
|
ejpbruel
:
review+
|
Details | Diff | Splinter Review |
27.77 KB,
patch
|
gkw
:
feedback+
decoder
:
feedback+
|
Details | Diff | Splinter Review |
This removes a bunch of gnarly hoisting code in BindVarOrConst that is never executed. Passes jit and reftests, trying servering now...
Attachment #644014 -
Flags: review?(ejpbruel)
Assignee | ||
Comment 1•12 years ago
|
||
Removing old frontend cruft inevitably awakens hidden demons; I humbly request some fuzzing.
Attachment #644027 -
Flags: feedback?(gary)
Attachment #644027 -
Flags: feedback?(choller)
Comment 2•12 years ago
|
||
Comment on attachment 644027 [details] [diff] [review] combined patch for fuzzing (applies to cset f08c782d0112) feedback+ from fuzzing for an hour or two - no biggies found from me... yet. ;-)
Attachment #644027 -
Flags: feedback?(gary) → feedback+
Comment 3•12 years ago
|
||
Comment on attachment 644027 [details] [diff] [review] combined patch for fuzzing (applies to cset f08c782d0112) Same here, nothing found so far. I'll leave it running over night though :)
Attachment #644027 -
Flags: feedback?(choller) → feedback+
Assignee | ||
Comment 4•12 years ago
|
||
Awesome, thanks guys.
Updated•12 years ago
|
Whiteboard: [js:t]
Comment 5•12 years ago
|
||
Comment on attachment 644014 [details] [diff] [review] rm Review of attachment 644014 [details] [diff] [review]: ----------------------------------------------------------------- Afaict, you are right, and this is indeed all that code does. Thank you for putting in a comment as well :-)
Attachment #644014 -
Flags: review+
Updated•12 years ago
|
Attachment #644014 -
Flags: review?(ejpbruel)
Assignee | ||
Comment 6•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/3c72277eced5
Target Milestone: --- → mozilla17
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/3c72277eced5
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•