Open Bug 831773 Opened 11 years ago Updated 2 years ago

IonMonkey: rerun GVN after LICM (or subset)

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect

Tracking

()

People

(Reporter: h4writer, Unassigned)

References

Details

This is visible in raytrace.js:709 with almost all functions inlined. 

LICM hoists a lot of definitions out of the loop. Afterwards there are some definitions that could get removed again by running GVN again.
Off-course we could be smarter and only check hoisted MIR's and we only have to compare in the current block. So could be a really small pass.

Shows a 1.5% improvement on raytrace by running full GVN again after LICM
Blocks: 768745
Assignee: general → nobody
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.