Closed
Bug 619849
Opened 14 years ago
Closed 14 years ago
JM: shrink JITScript on 64-bit platforms
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| blocking2.0 | --- | betaN+ |
People
(Reporter: n.nethercote, Assigned: n.nethercote)
References
Details
(Whiteboard: [cib-memory], fixed-in-tracemonkey)
Attachments
(1 file)
|
2.67 KB,
patch
|
dvander
:
review+
|
Details | Diff | Splinter Review |
JITScript is sub-optimally packed on 64-bit platforms. The attached patch reduces its size from 232 to 200 bytes. On a 20-tab cad-comics browser run, this saves 357,102 bytes. It's a small win, but it's a dead-simple patch.
Attachment #498271 -
Flags: review?(dvander)
|
Assignee | |
Updated•14 years ago
|
blocking2.0: --- → ?
|
|
Assignee | |
Updated•14 years ago
|
Whiteboard: [cib-memory]
|
|
||
Updated•14 years ago
|
Attachment #498271 -
Flags: review?(dvander) → review+
|
|
Assignee | |
Comment 1•14 years ago
|
||
A |bool singleStepMode| field was added since I created the patch, I squeezed that into the bottom bit of |nCallSites|. http://hg.mozilla.org/tracemonkey/rev/35697ebafa6c
Whiteboard: [cib-memory] → [cib-memory], fixed-in-tracemonkey
|
||
Updated•14 years ago
|
blocking2.0: ? → beta9+
|
||
Comment 2•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/35697ebafa6c
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
As per today's meeting, beta 9 will be a time-based release. Marking these all betaN+. Please move it back to beta9+ if you believe it MUST be in the next beta (ie: trunk is in an unshippable state without this)
No longer blocks: JaegerShrink
blocking2.0: beta9+ → betaN+
|
||
Updated•14 years ago
|
Blocks: JaegerShrink
Fixing fields my automated script accidentally blanked. Apologies for the bugspam
You need to log in
before you can comment on or make changes to this bug.
Description
•