Closed
Bug 1040194
Opened 11 years ago
Closed 11 years ago
Eliminate dead ResumePoint operands should not remove escaped objects.
Categories
(Core :: JavaScript Engine: JIT, defect)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
mozilla34
People
(Reporter: nbp, Assigned: nbp)
References
Details
Attachments
(1 file)
|
1.93 KB,
patch
|
h4writer
:
review+
|
Details | Diff | Splinter Review |
Scalar replacement is removing all uses and replace all loads which are using the object. In case of bailout, we still want to recover the object for baseline, even if we are able to remove all the uses from Ion.
|
Assignee | |
Comment 1•11 years ago
|
||
This patch change the logic to always keep recoverOnBailout instruction in
resume point operands. This is to work around all the uses which got
removed with the previous optimizations such as scalar replacement.
|
|
Assignee | |
Updated•11 years ago
|
Attachment #8458126 -
Flags: review?(hv1989)
|
||
Updated•11 years ago
|
Attachment #8458126 -
Flags: review?(hv1989) → review+
|
|
Assignee | |
Comment 2•11 years ago
|
||
|
||
Comment 3•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in
before you can comment on or make changes to this bug.
Description
•