Closed Bug 1326213 Opened 5 years ago Closed 5 years ago

Unneeded abort at "Cannot inspect prototype chain."

Categories

(Core :: JavaScript Engine: JIT, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: h4writer, Assigned: h4writer)

Details

Attachments

(1 file)

Saw this abort happening on a benchmark. Looking at the code I think we can just not do the optimization and go on. No need to disable compiling the script IIUC.
Attached patch PatchSplinter Review
Assignee: nobody → hv1989
Attachment #8822438 - Flags: review?(jdemooij)
Comment on attachment 8822438 [details] [diff] [review]
Patch

Review of attachment 8822438 [details] [diff] [review]:
-----------------------------------------------------------------

Great catch!
Attachment #8822438 - Flags: review?(jdemooij) → review+
Priority: -- → P1
Pushed by hv1989@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c6a6571e7468
IonMonkey: Don't abort when trying to speculate on unstable proto, r=jandem
https://hg.mozilla.org/mozilla-central/rev/c6a6571e7468
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.