IonMonkey OOM Testing: Assertion failure: frame.stackDepth() == opinfo->stackDepth, at js/src/methodjit/Compiler.cpp:2146

RESOLVED FIXED in mozilla15

Status

()

RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: decoder, Assigned: jandem)

Tracking

({assertion, testcase})

Other Branch
mozilla15
x86_64
Linux
assertion, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
The following command asserts on ionmonkey revision 8c54899dae82 (dbg build):

js  -e 'const libdir = "js/src/jit-test/lib/";' -A 14620 -f js/src/jit-test/tests/sunspider/check-mont.js
(Reporter)

Comment 1

7 years ago
Still valid with ballast patch, new -A value for revision 88ea2e529609 is 10450.
(Assignee)

Comment 2

7 years ago
Created attachment 625953 [details] [diff] [review]
Patch for inbound

JM bug. The patch propagates OOM from TypeObject::getProperty in jsop_getgname and jsop_setgname. This matches jsop_getprop and jsop_setprop.
Assignee: general → jdemooij
Status: NEW → ASSIGNED
Attachment #625953 - Flags: review?(bhackett1024)
(Assignee)

Updated

7 years ago
Summary: IonMonkey: OOM Testing: Assertion failure: frame.stackDepth() == opinfo->stackDepth, at js/src/methodjit/Compiler.cpp:2146 → OOM Testing: Assertion failure: frame.stackDepth() == opinfo->stackDepth, at js/src/methodjit/Compiler.cpp:2146
(Reporter)

Comment 3

7 years ago
Thanks for fixing it anyway :)
Summary: OOM Testing: Assertion failure: frame.stackDepth() == opinfo->stackDepth, at js/src/methodjit/Compiler.cpp:2146 → JS OOM Testing: Assertion failure: frame.stackDepth() == opinfo->stackDepth, at js/src/methodjit/Compiler.cpp:2146
Summary: JS OOM Testing: Assertion failure: frame.stackDepth() == opinfo->stackDepth, at js/src/methodjit/Compiler.cpp:2146 → IonMonkey OOM Testing: Assertion failure: frame.stackDepth() == opinfo->stackDepth, at js/src/methodjit/Compiler.cpp:2146
Attachment #625953 - Flags: review?(bhackett1024) → review+
https://hg.mozilla.org/mozilla-central/rev/65b0d19ddb92
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.