Closed Bug 780973 Opened 10 years ago Closed 10 years ago

IonMonkey: Try to enter outer loops rather than inner loops via OSR

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: jandem, Assigned: jandem)

References

(Blocks 1 open bug)

Details

(Whiteboard: [ion:p1:fx18])

Attachments

(1 file)

Patch
15.64 KB, patch
dvander
: review+
bhackett1024
: review+
Details | Diff | Splinter Review
Attached patch Patch — — Splinter Review 
This patch changes the recompile checks inserted by JM and Ion to use a slightly higher threshold for inner loops. The result is that we are more likely to enter outer loops instead of inner loops via OSR.

This closes about half the gap with JM+TI on ss-fannkuch (the rest seems to be compilation time, we are now faster than JM+TI on the modified benchmark in bug 777572), fixes the micro-benchmark in bug 779860 comment 0 and is a small v8-richards win.
Attachment #649764 - Flags: review?(dvander)
Attachment #649764 - Flags: review?(bhackett1024)
Whiteboard: [ion:p1:fx18]
Attachment #649764 - Flags: review?(dvander) → review+
Attachment #649764 - Flags: review?(bhackett1024) → review+
https://hg.mozilla.org/projects/ionmonkey/rev/5e461fb2250a
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.