Closed
Bug 512423
Opened 17 years ago
Closed 16 years ago
nanojit: inline small RegAlloc methods
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| status1.9.2 | --- | beta1-fixed |
People
(Reporter: n.nethercote, Assigned: n.nethercote)
References
Details
(Whiteboard: fixed-in-tracemonkey)
Attachments
(1 file)
|
6.07 KB,
patch
|
edwsmith
:
review+
|
Details | Diff | Splinter Review |
RegAlloc has a number of small methods that are called lots of times during register allocation, which makes them ideal for inlining. The attached patch does this inlining. It also removes an unnecessary level of indentation in RegAlloc.h.
I don't see a SS speedup but fewer instructions are being executed and the executable size is 320 bytes smaller (on Mac) so it seems worthwhile.
Attachment #396396 -
Flags: review?(edwsmith)
|
||
Updated•17 years ago
|
Attachment #396396 -
Flags: review?(edwsmith) → review+
|
|
Assignee | |
Comment 1•17 years ago
|
||
Whiteboard: fixed-in-tracemonkey
|
||
Comment 2•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
|
||
Comment 3•16 years ago
|
||
status1.9.2:
--- → beta1-fixed
Flags: wanted1.9.2+
You need to log in
before you can comment on or make changes to this bug.
Description
•