Closed Bug 893732 Opened 6 years ago Closed 6 years ago

Assertion failure: a.isGeneralReg(), at ../ion/shared/CodeGenerator-shared-inl.h:31

Categories

(Core :: JavaScript Engine, defect, critical)

x86
Linux
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: decoder, Assigned: bhackett)

References

(Blocks 2 open bugs)

Details

(Keywords: assertion, regression, testcase, Whiteboard: [jsbugmon:update])

Attachments

(2 files)

The following testcase asserts on mozilla-central revision 18467a85acf6 (run with --fuzzing-safe --ion-eager):


var arr = new Int8Array(100);
function f(a) {
  for(var i=0; i<30; i++) {
    x = a[85.3];
  }
}
f(arr);
Whiteboard: [jsbugmon:update,bisect]
Whiteboard: [jsbugmon:update,bisect] → [jsbugmon:update]
JSBugMon: Bisection requested, result:
autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   http://hg.mozilla.org/mozilla-central/rev/0670cdaf7e9c
user:        Brian Hackett
date:        Thu Jul 11 15:08:26 2013 -0600
summary:     Bug 891400 - Improve pattern matching on static typed array accesses, r=jandem.

This iteration took 0.973 seconds to run.
Needinfo from Brian based on comment 2 :)
Flags: needinfo?(bhackett1024)
Blocks: 891400
Keywords: regression
Duplicate of this bug: 894727
Duplicate of this bug: 893726
Attached patch patchSplinter Review
convertShiftToMaskForStaticTypedArray can now produce non-int32 indexes, and there was no type policy in place to ensure these were converted to integers.
Assignee: general → bhackett1024
Attachment #777846 - Flags: review?(jdemooij)
Flags: needinfo?(bhackett1024)
Attachment #777846 - Flags: review?(jdemooij) → review+
https://hg.mozilla.org/mozilla-central/rev/d7af6a55966c
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in before you can comment on or make changes to this bug.