Closed
Bug 456588
Opened 16 years ago
Closed 11 years ago
localize var scope to nearest block enclosing def and uses if def dominates uses
Categories
(Core :: JavaScript Engine, defect, P2)
Core
JavaScript Engine
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: brendan, Assigned: bhackett1024)
References
Details
(Whiteboard: DUPEME)
Attachments
(1 file, 1 obsolete file)
|
(deleted),
patch
|
Details | Diff | Splinter Review |
This is needed to handle union types of (undefined, double), (undefined, string), etc. in TM.
/be
|
||
Updated•16 years ago
|
Severity: normal → enhancement
|
Reporter | |
Updated•16 years ago
|
|
|
Reporter | |
Updated•16 years ago
|
Target Milestone: mozilla1.9.1 → mozilla1.9.2a1
|
|
Reporter | |
Updated•15 years ago
|
Target Milestone: mozilla1.9.2a1 → mozilla1.9.3
|
|
Reporter | |
Updated•15 years ago
|
Priority: P2 → P1
Target Milestone: mozilla1.9.3 → mozilla1.9.3a5
|
|
Reporter | |
Comment 1•15 years ago
|
||
Needs cleanup and testing/tests.
/be
|
||
Comment 2•15 years ago
|
||
(In reply to comment #1)
> Created an attachment (id=438690) [details]
> draft patch
>
> Needs cleanup and testing/tests.
>
> /be
Ready for fuzz testing?
|
|
Reporter | |
Comment 3•15 years ago
|
||
Attachment #438690 -
Attachment is obsolete: true
|
|
Reporter | |
Comment 4•14 years ago
|
||
IIRC bhackett's JM analysis does this. Plus, we want to get into better front end and high-level intermediate reps (Shu's SSA in one pass work). So this should not be rushed in terms of the current def/use chains.
/be
Status: ASSIGNED → NEW
Target Milestone: mozilla1.9.3a5 → ---
|
|
Reporter | |
Comment 5•14 years ago
|
||
I think bhackett has this one, possibly even under another bug.
/be
Assignee: brendan → bhackett1024
Whiteboard: DUPEME
|
||
Comment 6•11 years ago
|
||
Brian: is this bug still relevant?
Flags: needinfo?(bhackett1024)
Priority: P1 → P2
|
Assignee | |
Comment 7•11 years ago
|
||
I think the SSA analysis done by Ion (and before, by JM+TI) captures this.
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(bhackett1024)
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•