Closed Bug 756777 Opened 10 years ago Closed 10 years ago
Monkey: Assertion failure: addr % Cell::Cell Size == 0, at ../../gc/Heap .h:832
The attached testcase asserts on ionmonkey revision 890dd17b4187 (run with --ion -n -m --ion-eager).
Works for me at revision 9602aebd7e43a27f1675ca8313b80045c383c7fb.
I'm still seeing this on the fuzzer, let's see if it still repros.
Whiteboard: [jsbugmon:update] → [jsbugmon:update,reconfirm]
This seemed to sketchy to just be fixed, at first I thought it was bug 756235 but reproduces with that applied too. Turns out it's similar though. LDefVar wasn't marked as a call, so LSRA gave it safepoint regs, which of course weren't spilled. To make it a call we also have to make it use the call temporary regs.
Assignee: general → dvander
Status: NEW → ASSIGNED
Attachment #626270 - Flags: review?
10 years ago
Attachment #626270 - Flags: review? → review?(sstangl)
Attachment #626270 - Flags: review?(sstangl) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.