Closed
Bug 599472
Opened 14 years ago
Closed 6 years ago
Vector.<Number> valueToAtom uses non SSE doubleToAtom
Categories
(Tamarin Graveyard :: Virtual Machine, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: wsharp, Unassigned)
References
Details
(Whiteboard: PACMAN)
Atom valueToAtom(double value) const { return core()->doubleToAtom(value); } Investigate whether a runtime sse2 check here and calling doubleToAtom_sse2 helps performance. Therre are a few other hardcoded integer_d and/or doubleToAtom calls in the code such as AvmCore::integer, etc. I've seen the Vector.<Number> one high on a VTUNE profile but not the others.
Reporter | ||
Updated•14 years ago
|
Whiteboard: PACMAN
Updated•14 years ago
|
Blocks: vector-tracker
Comment 1•13 years ago
|
||
Arguably, the JIT should not use the non-native get or helpers that call valueToAtom or atomToValue(), but instead inline the value coersion or boxing code. In either case, the existing JIT code will use the faster doubleToAtom_sse2, or coerce logic. If this is pursued, its important not to change the order of side effects. See bug 687338.
Comment 2•6 years ago
|
||
Tamarin isn't maintained anymore. WONTFIX remaining bugs.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•