IonMonkey: Assertion failure: !entered && i < mLength, at ./dist/include/js/Vector.h:373

RESOLVED FIXED

Status

()

Core
JavaScript Engine
--
major
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: decoder, Assigned: jandem)

Tracking

(Blocks: 2 bugs, {assertion, testcase})

Other Branch
x86_64
Linux
assertion, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
The following testcase asserts on ionmonkey revision c34398f961e7 (run with --ion -n -m), tested on 64 bit:


function nsieve(m, isPrime){
   for (i=2; i<=m; (g.args[i])){
         for (k=i+i; k<=m; k+=i) isPrime[k] = false;
      break;
   }
}
var ret = 0;
    for (var i = 1; i <= 3; i++ ) {
        var m = (1<<i)*10000;
        var flags = Array(m+(0XeE0c  ));
        ret += nsieve(m, flags);
}
(Assignee)

Comment 1

5 years ago
Created attachment 595148 [details] [diff] [review]
Patch

Missing some cases for for-loops in IonBuilder::isLoop. Patch fixes the test and passes jit-tests. Surprising that this didn't cause jit-test failures.
Assignee: general → jdemooij
Status: NEW → ASSIGNED
Attachment #595148 - Flags: review?(dvander)
Attachment #595148 - Flags: review?(dvander) → review+
(Assignee)

Comment 2

5 years ago
http://hg.mozilla.org/projects/ionmonkey/rev/6916659b5ba5
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Reporter)

Comment 3

4 years ago
A testcase for this bug was automatically identified at js/src/jit-test/tests/ion/bug724999.js.
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.