Add AS3 typed SunSpider tests

VERIFIED FIXED

Status

VERIFIED FIXED
11 years ago
10 years ago

People

(Reporter: brbaker, Assigned: brbaker)

Tracking

({flashplayer})

unspecified
x86
Windows XP
flashplayer
Bug Flags:
in-testsuite +
flashplayer-qrb +

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Assignee)

Description

11 years ago
The current typed sunspider tests do not use classes or vectors. We should add a version that is AS3 typed with classes and another set that also uses vectors. Once tamarin-tracing has vector support we can merge the "as3typed" and "as3typedvector" together.
(Assignee)

Comment 1

11 years ago
Created attachment 324295 [details] [diff] [review]
First pass

There is probably a little more work that can be done in the as3typed/s3d-raytracer (moving to a custom vector class for transformation calculations). There is still more work that can be done in the as3typedvector. Mainly trying to get as3typedvector/s3d-cube working with vectors.
Attachment #324295 - Flags: review?(dschaffe)
(Assignee)

Comment 2

11 years ago
Created attachment 326291 [details] [diff] [review]
Update the "typed" sunspider tests to fully AS3 typed

This also adds in the vector versions of the typed tests but they will not run until vector support is added to TT. The typedvector tests are marked to be skipped.
Attachment #324295 - Attachment is obsolete: true
Attachment #324295 - Flags: review?(dschaffe)
(Assignee)

Comment 3

11 years ago
Created attachment 326742 [details]
Updated to wrap top level functions in a package

Patch is zipped.
Attachment #326291 - Attachment is obsolete: true
Attachment #326742 - Flags: review?(dschaffe)

Comment 4

11 years ago
Comment on attachment 326742 [details]
Updated to wrap top level functions in a package

adding a new patch.  it's the same with updates to s3d-cube.
Attachment #326742 - Attachment is obsolete: true
Attachment #326742 - Flags: review?(dschaffe) → review-

Comment 5

11 years ago
Created attachment 326910 [details] [diff] [review]
patch to sunspider tests
Attachment #326910 - Flags: review?(brbaker)
(Assignee)

Updated

11 years ago
Attachment #326910 - Flags: review?(brbaker) → review+

Comment 6

11 years ago
We may want to add an additional comment to the source files (below the licensing comment) pointing out that this is the optimized-for-AS3 version of the Sunspider test.

Updated

10 years ago
Assignee: nobody → brbaker
Flags: flashplayer-qrb+

Updated

10 years ago
Flags: in-testsuite?
(Assignee)

Comment 7

10 years ago
changeset:   594:7062ea2d004f
user:        Dan Schaffer <dschaffe@adobe.com>
date:        Thu Jun 26 11:43:16 2008 -0400
summary:     update sunspider/typed performance tests to use classes and vectors.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Flags: in-testsuite? → in-testsuite+
Keywords: flashplayer
Resolution: --- → FIXED
(Assignee)

Updated

10 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.