654498 - linux 32bit shells fail self validation for some of the jsbench performance testcases

Status: RESOLVED WONTFIX

(Tamarin Graveyard :: Virtual Machine, defect, P3)

Description

[Brent Baker]

linux 32bit shells fail self validation for some of the jsbench performance testcases.

jsbench/LUFact
jsbench/Moldyn
jsbench/Raytracer

For example:
The testcase jsbench/LUFact.as does matrix decompostion and then calculates norm and compares it against a reference norm, if the values are close it passes validation.

This testcase will produce a computed norm of 5.167473096289751 on all platforms except for linux32. 
Computed Norm Res = 5.872537646878168 (linux 32-bit)
Computed Norm Res = 5.167473096289751 (linux 64-bit)

Matching reference norm:
Computed Norm Res = 5.167473096289751
platforms: win32/64, mac32/64, linux64, android-arm
run mode: hybrid, interp, Ojit

Failing to match reference norm:
linux32:
hybrid/Ojit: 5.872537646878168, this is actually close enough to PASS the validation, but does not match all other platforms
interp: 6.3198285975877715

injection: NO, I have tested with old shells (coral and argo) and the behave in the same manner, producing the exact same results as above.

Comment 1

[Brent Baker]

It "may" be possible that this is related to bug# 521245 (Differences in some ecma3/Math tests when running 64bit vs 32bit shells), but I do not see any calls to any builtin math functions so does not seem likely.

Comment 2

[Brent Baker]

Attachment: jsbench self validation

Dan if you want to review this patch, it enables the self validation for some of the left over jsbench tests, but they do not validate properly on lin32 (essence of this bug).

Comment 3

[Brent Baker]

Comment on attachment 529762 [details] [diff] [review]
jsbench self validation

Note from bug# 526234, where this patch originally came from:
- important change here is the moldyn now actually works, previously was
generating bogus calculations (NaN), now works properly

Comment 4

[Brent Baker]

brbaker: please review patch and land this in tr. (note to self)

Comment 5

[Sylvestre Ledru [:Sylvestre]]

Tamarin is a dead project now. Mass WONTFIX.

Comment 6

[Sylvestre Ledru [:Sylvestre]]

Tamarin isn't maintained anymore. WONTFIX remaining bugs.