Closed Bug 707158 Opened 13 years ago Closed 13 years ago

Interpreter and C++ code is not calling machine float primitive functions

Categories

(Tamarin Graveyard :: Interpreter, defect, P3)

Product:
Tamarin Graveyard
Component:
Interpreter
Platform:
x86
All
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED DUPLICATE of bug 704097
Milestone:
Q2 12 - Cyril

People

(Reporter: brbaker, Unassigned)

References

Details

With changes introduced in tr-float changeset 6989:35967134e840, there is now a visible difference between jitted code and interpreted code with regards to some of the implemented float intrinsics.

changeset:   6989:35967134e840
user:        William Maddox <wmaddox@adobe.com>
date:        Tue Nov 29 18:40:08 2011 -0800
summary:     Bug 703604 - Inline those float methods that correspond to inlined math methods

At the moment that following test media will produce different results between jit and interp:
as3/Types/Float/reciprocal.as
as3/Types/Float/rsqrt.as
as3/Types/Float/nonstrict/reciprocal.as
as3/Types/Float/nonstrict/rsqrt.as

There is also visible differences in these tests between release and release-debugger builds when jitted. The release-debugger builds do NOT show a difference between jitted code and interpreted code, is it possible that this configuration does not inline these methods?
Flags: in-testsuite+
Flags: flashplayer-triage+
Flags: flashplayer-qrb+
Flags: flashplayer-bug-
Mmm, interesting observation about release-debugger - was that with or without -Dnodebugger?
Priority: -- → P3
Target Milestone: --- → Q2 12 - Cyril
Probably a dup of bug #704097?
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
Status: RESOLVED → VERIFIED
changeset: 7048:16505ab075f8
user:      Brent Baker <brbaker@adobe.com>
summary:   Clean up expected failures for fixed bugs 706454 and 703259.

http://hg.mozilla.org/tamarin-redux/rev/16505ab075f8
changeset: 7049:66054357a2a6
user:      Brent Baker <brbaker@adobe.com>
summary:   Bug 707158: failures should currently only be tracked for release and debug builds

http://hg.mozilla.org/tamarin-redux/rev/66054357a2a6
You need to log in before you can comment on or make changes to this bug.