If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

changeset 311:1342ce084f37 is causing crashes in debug vm

RESOLVED FIXED

Status

Tamarin
Tracing Virtual Machine
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: Brent Baker, Assigned: Steven Johnson)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
There is an assert being thrown when running the debug or debug-debugger vm in 145 acceptance tests. Here is a sample:

e4x/Expressions/e11_1_3.as
ecma3/Array/e15_4_4_13.as
ecma3/ObjectObjects/class_001.as

To see the entire list check the buildbot status @ http://tamarin-builds.mozilla.org/tamarin-tracing/waterfall


Assertion failed: "((isconst()))" ("e:\\buildbot\\slave-tamarin-tracing-windows\
\tamarin-tracing\\nanojit\\LIR.h":208)
avmplus crash: exception 0x80000003 occurred
Writing minidump crash log to avmplusCrash.dmp
(Reporter)

Comment 1

10 years ago
All of these tests pass when running the 309 vm.
(Reporter)

Updated

10 years ago
Summary: changeset 311:1342ce084f37 is causes crashes in debug vm → changeset 311:1342ce084f37 is causing crashes in debug vm
(Assignee)

Comment 2

10 years ago
this is a debug-only failure but I have a fix ready to land.
(Assignee)

Comment 3

10 years ago
Created attachment 308669 [details] [diff] [review]
Patch
Assignee: nobody → stejohns
Status: NEW → ASSIGNED
Attachment #308669 - Flags: review?(brbaker)
(Reporter)

Comment 4

10 years ago
Comment on attachment 308669 [details] [diff] [review]
Patch

Tests are passing with the patch. Currently only about  1/10 of the way through the tests but they are all passing now instead of throwing an assert, and it will take a while to complete on the current machine.
Attachment #308669 - Flags: review?(brbaker) → review+
(Reporter)

Comment 5

10 years ago
Patch submitted as 318:3a3db3420551
(Assignee)

Comment 6

10 years ago
Should this be marked FIXED?
(Reporter)

Updated

10 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.