Javascript test case testTypedArrays.js failed on sparc with typed array and TI

RESOLVED FIXED in mozilla10

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Leon Sha, Assigned: Leon Sha)

Tracking

unspecified
mozilla10
Sun
Solaris
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

6 years ago
Multiple cases failed on sparc with typed array and TI. This is the regression from Bug 693144.
(Assignee)

Comment 1

6 years ago
Created attachment 569285 [details] [diff] [review]
patch

Change lengthOffset() back to offset of Value.
Assignee: general → leon.sha
Status: NEW → ASSIGNED
Attachment #569285 - Flags: review?(bhackett1024)
Attachment #569285 - Flags: review?(bhackett1024) → review+
(Assignee)

Comment 2

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/31de0fd80b15

Comment 3

6 years ago
https://hg.mozilla.org/mozilla-central/rev/31de0fd80b15
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
(Assignee)

Comment 4

6 years ago
Created attachment 569852 [details] [diff] [review]
Patch part 2

I thought TypedArray::lengthOffset() only used in methodjit.
Attachment #569852 - Flags: review?(bhackett1024)
Attachment #569852 - Flags: review?(bhackett1024) → review+
(Assignee)

Comment 5

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/38225e65bf70
https://hg.mozilla.org/mozilla-central/rev/38225e65bf70
You need to log in before you can comment on or make changes to this bug.