The default bug view has changed. See this FAQ.

Sync FrameState entries based on tracker

RESOLVED FIXED in mozilla10

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: bhackett, Unassigned)

Tracking

(Blocks: 1 bug)

unspecified
mozilla10
x86
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 570360 [details] [diff] [review]
patch

As described in 685358 comment 4, the FrameState's current behavior of scanning all entries in OOL and inline sync paths can cost a lot of time.  This behavior was introduced with TI due to the need to keep the entire stack synced when making calls.  Given this requirement, it is faster on large scripts to scan tracked entries instead.  The attached patch reduces the sqlite compile time in bug 687127 from ~5250ms to ~4900ms.
Attachment #570360 - Flags: review?(dvander)
Attachment #570360 - Flags: review?(dvander) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/1b4d0987b18d
https://hg.mozilla.org/mozilla-central/rev/1b4d0987b18d
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in before you can comment on or make changes to this bug.