Closed
Bug 1141645
Opened 10 years ago
Closed 8 years ago
Add a RegExp object Alias Set and Recover RegExp global state.
Categories
(Core :: JavaScript Engine: JIT, defect)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: nbp, Unassigned)
References
Details
Currently regexp cannot be moved or removed, even with recover instructions as highlighted by Bug 1132128 test cases because the global RegExp object is modified with the result of the match.
To work around this issue we would have to add a new alias set for RegExp instructions, and use a modified Alias Analysis which will inform us if the instruction is post-dominated, by another RegExp instruction.
Also, to recover properly the RegExp results, we would have to optionally account from the RegExp object state as part of the resume point, if one of the RegExp instruction is removed.
|
||
Comment 1•9 years ago
|
||
I think we can close this bug, since bug 1207922 selfhost the lastIndex part. (It is not done by RegExpExec/RegExpTest, now RegExpMatcher, anymore) In other words we don't need any specific ion code to handle the .lastIndex.
|
||
Comment 2•8 years ago
|
||
Closing as per comment 1.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•