Closed
Bug 783272
Opened 12 years ago
Closed 12 years ago
IonMonkey: Remove code to eagerly spill phis if there's no register available
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: jandem, Assigned: jandem)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
1.23 KB,
patch
|
dvander
:
review+
|
Details | Diff | Splinter Review |
Bug 675381 added some code to prevent blocked register allocation at phis. This hurts the loop in 3d-cube's DrawLine; the code Ion currently generates for the loop is atrocious. The patch reduces the instruction count for that loop from 43 to 29 and reduces the benchmark time from 12.2 to 11.7 ms. I will fuzz this for a few hours before landing, just in case I'm missing something, but I'm pretty sure this patch is safe now.
Attachment #652449 -
Flags: review?(dvander)
Updated•12 years ago
|
Attachment #652449 -
Flags: review?(dvander) → review+
Assignee | ||
Comment 1•12 years ago
|
||
https://hg.mozilla.org/projects/ionmonkey/rev/d794f23798f4 This didn't turn out to be a large 3d-cube win on AWFY, but it helped ss-nsieve, navier-stokes and gaussian-blur.
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
•