700642 - pre and postfix ++ and -- on float4 values incorrect in hybrid mode

Status: RESOLVED WONTFIX

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

Description

[Brent Baker]

I am not 100% sure why this is but the following code does not produce the correct value for the 'w' component of the float4 value. This only happens in hybrid mode, -Dinterp and -Ojit produce the correct result.

This is happening on ARM-android

Code:
var flt4:float4 = new float4(1.125f);
print(++flt4);

Output in hybrid mode:
2.125,2.125,2.125,1.125

Output expected and with -Ojit and -Dinterp:
2.125,2.125,2.125,2.125


Testmedia:
as3/Types/Float4/flt4_5_1_6.as
as3/Types/Float4/flt4_5_1_7.as
as3/Types/Float4/flt4_5_1_8.as
as3/Types/Float4/flt4_5_1_9.as

Comment 1

[Brent Baker]

Injection point:
http://asteam.corp.adobe.com/hg/tr-float/rev/e542ff003f8c

changeset:   6796:e542ff003f8c
user:        William Maddox <[e-mail]>
date:        Mon Nov 07 13:52:48 2011 -0800
summary:     Don't use callee-saved FP registers on ARM.

Comment 2

[Brent Baker]

Bill to throw more tests at this... bug #702598 was fixed which allowed the Float4/float4Operations.as file run with debug builds on android and now I am seeing most of these tests fail with what appears to be an incorrect w component, the sign of the w component is reversed.

Comment 3

[William Maddox]

Fix botch in changeset:   6796:e542ff003f8c

http://asteam.macromedia.com/hg/tr-float/rev/effd50dbb34c

Comment 4

[William Maddox]

(In reply to Brent Baker from comment #2)

The Android ++ issue should be fixed.  Can you verify?

The Float4/float4Operations.as test case fails on ARM Linux as well,
and is still failing.  In the injection build for the ++ issue, this
test case wasn't being run, due to bug 702598, so I don't see the
evidence that it is the same issue.  I suspect it should have its
own bug.

Comment 5

[William Maddox]

Opened bug 703512 for the float4Operations issues.

Comment 6

[Tamarin Bot]

changeset: 6966:d865c4823126
user:      Brent Baker <[e-mail]>
summary:   Bug 702598 and 702632 and now fixed, remove from failconfig.txt

http://hg.mozilla.org/tamarin-redux/rev/d865c4823126

Comment 7

[Sylvestre Ledru [:Sylvestre]]

Tamarin isn't maintained anymore. WONTFIX remaining bugs.