TM: Remove _frags and all related fragment mapping code from Fragmento

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
10 years ago
7 years ago

People

(Reporter: gal, Assigned: gal)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

10 years ago
The main tracer already uses a separate (ip, globalShape) => Fragment mapping. Fragmento's mapping is a binary search and is slow and not powerful enough. Only the regexp jit uses it at this point, and needs a separate fragmento (and code cache) due to that.

The attached patch does the following:
- gut fragmento, remove ip -> fragment mapping and lot of dead statistics code
- make regexp use its own regexp -> fragment mapping
- introduce js_InvalidateJITCache which will either immediately flush the JIT cache if possible (no native code is running, and no native code is on the stack), or it sets a flag to flush as soon as possible
- regexp now uses the main fragmento's code cache, but a separate lirbuf since we might compile regexps while recording a trace
(Assignee)

Comment 2

10 years ago
Created attachment 367684 [details] [diff] [review]
patch
Assignee: general → gal
Obsolete with the removal of tracejit.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.