Closed
Bug 557991
Opened 16 years ago
Closed 16 years ago
nanojit: remove dead code from CodeAlloc
Categories
(Core Graveyard :: Nanojit, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: n.nethercote, Assigned: n.nethercote)
References
Details
(Whiteboard: fixed-in-nanojit, fixed-in-tracemonkey, fixed-in-tamarin)
Attachments
(1 file)
|
7.77 KB,
patch
|
edwsmith
:
review+
|
Details | Diff | Splinter Review |
There's a lot of dead code in CodeAlloc. After removing it there was only one remaining call to containsPtr() and it is a simple function so I just inlined the call and removed the function as well.
I checked the patch on TM as well to make sure all these functions were dead in it too.
This blocks bug 506693 because that bug also modifies CodeAlloc, and removing this dead code means that fewer changes are needed there.
Attachment #437760 -
Flags: review?(edwsmith)
|
||
Comment 1•16 years ago
|
||
Comment on attachment 437760 [details] [diff] [review]
patch
Leftovers from an experimental jit-cache, fine to remove.
Attachment #437760 -
Flags: review?(edwsmith) → review+
|
|
Assignee | |
Comment 2•16 years ago
|
||
Whiteboard: fixed-in-nanojit
|
|
Assignee | |
Comment 3•16 years ago
|
||
Whiteboard: fixed-in-nanojit → fixed-in-nanojit, fixed-in-tracemonkey
|
|
||
Comment 4•16 years ago
|
||
Whiteboard: fixed-in-nanojit, fixed-in-tracemonkey → fixed-in-nanojit, fixed-in-tracemonkey, fixed-in-tamarin
|
||
Comment 5•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
|
||
Updated•12 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•